vegarsti commented on PR #8589:
URL: https://github.com/apache/arrow-rs/pull/8589#issuecomment-3401061415

   Yeah, I will give the arrow-ord partitioning a try. Some time this week! It 
seems like a good approach, thanks @tustvold!
   
   As for the feature flag, to me that seems a bit complicated - either the REE 
type should be supported or not, imo? Also, whether to put this in a feature 
flag would apply to the whole REE epic 
https://github.com/apache/arrow-rs/issues/3520, so that should be raised there.
   
   It would be great with some guidelines for the arrow-rs project with regard 
to the tradeoff between features and size/compile times, that might make it 
easier to point to in such discussions. I'm guessing opinions might vary a bit 
between maintainers as well.


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