Rich-T-kid opened a new pull request, #11868:
URL: https://github.com/apache/arrow-rs/pull/11868

   # Which issue does this PR close?
   
   - Closes #5208
   
   # Rationale for this change
   
   `Array::logical_nulls` clones the physical `NullBuffer` even when callers 
only need to read it.
   
   # What changes are included in this PR?
   
   Return `Cow<NullBuffer>`, borrowing existing buffers and owning computed 
logical masks. Update callers accordingly.
   
   # Are these changes tested?
   
   Adds coverage that physical masks are borrowed and computed masks are owned.
   
   # Are there any significant user-facing changes?
   
   Yes. `Array::logical_nulls` now returns `Option<Cow<'_, NullBuffer>>`.
   


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