benibus commented on PR #14753: URL: https://github.com/apache/arrow/pull/14753#issuecomment-1371273417
Yeah, makes sense. I can save the FieldRef/Path stuff for a later date. > we would have to decide whether to take into account outer-level nulls or not That was a point of uncertainty for me as well. My implementation of `FieldPath::FindAll` for `Table`s has child fields inherit their parent's validity via flattening, but the existing versions just use their underlying `ArrayData` directly (most likely to avoid allocating a new `Array`?), which would raise even more questions. -- 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]
