[GitHub] [arrow-rs] tustvold opened a new issue #1279: `ScalarBuffer::pad_nulls` Zeros Out Padded Data

[GitBox <gi...@apache.org>]

tustvold opened a new issue #1279:
URL: https://github.com/apache/arrow-rs/issues/1279


   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   
   `ScalarBuffer::pad_nulls` iterates backwards through the values, moving them into their final positions. It does this by swapping them with the zero-initialized value in the destination position.
   
   **Describe the solution you'd like**
   
   As far as I understand the specification, primitive arrays can have arbitrary values at null positions. Therefore it makes sense, at least to me, to simply copy the value to its destination and not zero out the source position.
   
   **Describe alternatives you've considered**
   
   The current logic is not incorrect and could remain
   


-- 
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-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-rs] alamb closed issue #1279: `ScalarBuffer::pad_nulls` Zeros Out Padded Data Unnecessarily

[GitBox <gi...@apache.org>]

alamb closed issue #1279:
URL: https://github.com/apache/arrow-rs/issues/1279


   


-- 
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-unsubscribe@arrow.apache.org

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