benibus commented on PR #34537: URL: https://github.com/apache/arrow/pull/34537#issuecomment-1481944005
@zeroshade @westonpace Looking at this a little deeper... do you know if `FieldPath::Get` is meant to be zero-copy? I'm not entirely sure if my suggestion to use `ChunkedArray::Flatten` here was appropriate since I believe it collapses top-level nulls. The `Get` implementations for `Array` and `RecordBatch` both use their underlying `ArrayData` exclusively without modifying/copying anything (regardless of nesting level). Achieving the same thing for `ChunkedArray`s would be more advanced, of course - but I can help with the implementation. -- 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]
