nevi-me opened a new pull request #8608: URL: https://github.com/apache/arrow/pull/8608
This changes a list datatype to use `Box<Field>` instead of `Box<DataType>`. This change is needed in order to make both Parquet and IPC roundtrips pass. The C++ implementation uses `Field`, as it allows for preservice list field names and nullability. There are some implementation details which we did not cover in this PR, and will work on as a follow-up, and these are: * Documenting the naming conventions of Arrow and Parquet on list names (Arrow C++ uses "item", Parquet has different compatibility options) * Fixing Parquet failing list tests that have been ignored (I'd like to do this separately, as this isn't a full fix) ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
