alamb opened a new pull request, #8866:
URL: https://github.com/apache/arrow-rs/pull/8866

   # Which issue does this PR close?
   
   - Closes https://github.com/apache/arrow-rs/issues/8657
   
   # Rationale for this change
   
   https://github.com/apache/arrow-rs/issues/8657 is a regression
   
   The check for "is this column nested" did not work correctly for Lists in 
Parquet, due to the [somewhat wacky way 
Lists](https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#lists)
 are encoded
   
   # What changes are included in this PR?
   
   1. Fix the bug
   2. Move the code into `ProjectionMask::without_nested_types`, mostly so I 
could write better tests for it
   3. Write a lot of tests 
   
   # Are these changes tested?
   Yes, both the reproducer from https://github.com/apache/arrow-rs/issues/8657 
and a bunch of tests are added
   
   # Are there any user-facing changes?
   
   There is a new API
   


-- 
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]

Reply via email to