EnricoMi commented on PR #44279: URL: https://github.com/apache/arrow/pull/44279#issuecomment-2515100175
Problem was that tests define an `ExtensionType` with storage type `DICTIONARY`, which caused an invalid cast, returning an empty pointer that was dereferenced. That has been fixes in a1ad4da. This means that user code that defines `ExtensionTypes` with other storage types than `DICTIONARY` may see unexpected behaviour. What would be a good `default` behaviour in `GetTypeForBuffer`? Return `Type::UINT8` to not realign those unknown data types? Or how can we tell from an unknown `ExtensionTypes` what to return by `GetTypeForBuffers`? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
