benibus opened a new pull request, #35197:
URL: https://github.com/apache/arrow/pull/35197

   ### Rationale for this change
   
   The current `FieldPath::Get` methods - when extracting nested child values - 
don't combine the child's null bitmap with higher-level parent bitmaps. While 
this is often preferable (it allows for zero-copy), there are cases where 
higher level "flattening" version is useful - e.g. when pre-processing sort 
keys for structs.
   
   ### What changes are included in this PR?
   
   - Adds `FieldPath::GetFlattened` methods alongside the existing  
`FieldPath::Get` overloads
   - Overhauls the `FieldPath` tests in an effort to improve coverage and 
generalize across the supported input types
   - Fixes a bug with nested `ArrayData` handling when the input array is a 
slice of a larger array (surfaced in the new tests)
   
   ### Are these changes tested?
   
   Yes (tests are included)
   
   ### Are there any user-facing changes?
   
   Yes, this adds methods to a public 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