zeroshade opened a new pull request, #520: URL: https://github.com/apache/arrow-go/pull/520
### Rationale for this change fixes #490 ### What changes are included in this PR? When using `PutDictionary` to write an arrow dictionary array to Parquet directly, we will now normalize the data type to a parquet physical storage type to avoid type problems. e.g. A dictionary with `int8` values will get cast to `int32` values before we process it so that all of the remaining encoding code can continue to be based on the physical storage type. This alleviates panics when using `pqarrow` to write Dictionary arrays. ### Are these changes tested? Yes, the unit tests are updated to include tests for int8/uint8/etc. as dictionary value types. ### Are there any user-facing changes? No, just fixing panics. -- 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]
