jecsand838 commented on PR #8349:
URL: https://github.com/apache/arrow-rs/pull/8349#issuecomment-3315542712

   > I'm pretty far out of my expertise on this PR, but hopefully the comments 
are still useful.
   
   @scovich Thank you so much for the review. Your comments were very useful 
and most of them should now be addressed in my latest commit.
   
   I'm wanting to follow-up on handling some of the redundancy you called out 
in your review. While I did clean up and abstracted what made sense, I think a 
full abstraction is part of a larger effort that needs to consider sparse 
unions as well.
   
   Let me know what you think whenever you get a chance!


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