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

   > I suggest we move the majority of the code in this PR to the DataFusion 
repo
   
   I covered this in the issue, but I wish for this code to live here so that 
it can be used within the arrow kernels, and benefit use-cases outside 
DataFusion.
   
   > still valid to have three variants
   
   I see no issue with DataFusion retaining its own row formats for these 
purposes, I'm personally unsure that these bit flips will add up to a 
significant overhead, but that's a somewhat ancillary discussion to be had down 
the line


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