nathaniel-d-ef opened a new pull request, #8550: URL: https://github.com/apache/arrow-rs/pull/8550
# Which issue does this PR close? Relates to: https://github.com/apache/arrow-rs/pull/8348 https://github.com/apache/arrow-rs/issues/4886 # Rationale for this change This PR completes the efforts of @jecsand838, adding dense union support to the encoder side of the crate, along with four other minor extensions of existing time-related encoding. Note: currently this PR is stacked behind https://github.com/apache/arrow-rs/pull/8546. Once that's merged this will be updated and will not include those changes. # What changes are included in this PR? - Dense union support for the writer - Tests # Are these changes tested? - A full round-trip test, reading in an existing union avro file and asserting that the output matches expectations - Unit tests covering new encoders. # Are there any user-facing changes? Crate not yet public -- 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]
