nathaniel-d-ef opened a new pull request, #8353: URL: https://github.com/apache/arrow-rs/pull/8353
# Which issue does this PR close? - Part of https://github.com/apache/arrow-rs/issues/4886 - Related to https://github.com/apache/arrow-rs/pull/8274 and https://github.com/apache/arrow-rs/pull/8298 # Rationale for this change This PR adds Map and Enum encoders to the arrow-avro crate writer, along with new benchmark tests for remaining types and round-trip tests. # What changes are included in this PR? New encoders: **Map** **Enum** Corresponding changes in support of these encoders in FieldEncoder and FieldPlan ## Additional round trip tests in `mod.rs` New tests follow existing file read pattern - simple_fixed - duration_uuid - nonnullable.impala.avro - decimals - enum ## Additional benchmark tests for data types - Utf8 - List<Utf8> - Struct - FixedSizeBinary16 - UUID - IntervalMonthDayNanoDuration - Decimal32(bytes) - Decimal64(bytes) - Decimal128(bytes) - Decimal128(fixed16) - Decimal256(bytes) - Map - Enum # Are these changes tested? Yes, additional complex type unit tests have been added for Map and Enum. The rest of the PR beyond the new types are tests themselves. All tests, new and existing, pass. # Are there any user-facing changes? n/a, arrow-avro crate is 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]
