etseidl commented on PR #6105:
URL: https://github.com/apache/arrow-rs/pull/6105#issuecomment-2251400644

   > The only thing I could think that we might want to explore is using a rust 
struct rather than a `Option<Vec<i64>` so it could be easier to use / document. 
I'll make a PR to explore what that looks like
   
   ❤️ 
   
   Yes, while I was reviewing my changes I was thinking the same thing. A 
histogram class that supports updates, unions, and concatenation.


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