scovich commented on issue #7119:
URL: https://github.com/apache/arrow-rs/issues/7119#issuecomment-2652217498

   > if you wanted to add a section documenting nested nullability, I'm sure 
that would be well received.
   
   For 
[`Array::is_null`](https://docs.rs/arrow/latest/arrow/array/trait.Array.html#method.is_null)
 and 
[`Array::logical_nulls`](https://docs.rs/arrow/latest/arrow/array/trait.Array.html#method.logical_nulls),
 something like:
   > WARNING: The logical nullability of a nested column is the union of 
logical nulls for all ancestors, which this method does not cover. See 
[`NullBuffer::union`](https://docs.rs/arrow/latest/arrow/buffer/struct.NullBuffer.html#method.union).
   
   
[`StructArray::into_parts`](https://docs.rs/arrow/latest/arrow/array/struct.StructArray.html#method.into_parts)
 should also be straightforward, something like:
   > WARNING: The returned columns may contain physically non-null but 
logically invalid values, due to logically null ancestors -- including but not 
limited to the returned `NullBuffer`, which itself may also be incomplete for 
the same reason.
   
   Maybe also 
[`StructArray::try_new`](https://docs.rs/arrow/latest/arrow/array/struct.StructArray.html#method.try_new)?
   > NOTE: Physically non-null values in child columns can become logically 
NULL if masked by the provided `NullBuffer` -- even if they are non-nullable.
   
   Anywhere else?


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to