tustvold commented on PR #6726:
URL: https://github.com/apache/arrow-rs/pull/6726#issuecomment-2473324150

   > so I can see the argument that by supporting this type of cast in arrow-rs 
it will evolve over time into a schema adapter / matching code
   
   Right, and then you're doing potentially non-trivial work per batch, 
matching columns, etc... as opposed to per-schema. For example DF's 
SchemaAdapter computes the mapping once and can then apply that to multiple 
batches.
   
   I guess what I am suggesting is that rather than baking this into the cast 
kernel, if we add a first-party schema adapter into arrow-rs. Potentially 
upstreaming the one already in DF with some modifications?


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