alamb commented on code in PR #8807:
URL: https://github.com/apache/arrow-rs/pull/8807#discussion_r2510724971


##########
arrow-buffer/src/util/bit_chunk_iterator.rs:
##########


Review Comment:
   Ah, I see
   
   yes I agree that some operation that moves the bits around in the `u64` word 
could give different results on different endianesses
   
   I think that is the case for the current bitwise operations too (they are 
supposed to be bitwise, not bit shuffling) 🤔 
   
   I will try and make a PR to update the docs to make this clearer



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