RyanMarcus commented on PR #6752:
URL: https://github.com/apache/arrow-rs/pull/6752#issuecomment-2484457349

   I know you just gave it as an example, but `take` would be significantly 
slower since it would require materializing an index array, which can't be 
trivially computed from the run ends (unlike for dictionaries, where the values 
are the indices). 
   
   I took a brief look at the other existing kernels and didn't see anything 
that looked promising. If you can point me to something specific I'm happy to 
take a look!
   
   I'm not currently familiar with MutableDataArray, let me take a look and get 
back to you. 
   
   One mitigating factor to the extra codegen is the fact that this is limited 
to the `cast` subcrate. We could also add an `arrow-cast-ree` or similar, if 
that's a concern. 


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