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

   > Hi @etseidl -- could you please retarget this PR at the `main` branch so I 
can merge it in?
   
   done
   
   > I also made [etseidl#1](https://github.com/etseidl/arrow-rs/pull/1) for 
your consideration (that make the level histogram a struct). We could also 
merge this PR as is and consider that as a follow on.
   
   I too created a `LevelHistogram` struct last night...yours is much more 
elegant 😄. I'm game to merge your PR into my branch before merging this into 
master. Or you can submit yours as a follow on PR. Whichever is easier for you.
   
   > Thank you for bearing with me here.
   
   Thank *you* for all the help and for the free Rust schooling 😄 


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