You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2023/06/29 16:57:13 UTC

[GitHub] [arrow-rs] alamb commented on pull request #4407: Remove Binary Dictionary Arithmetic Support

alamb commented on PR #4407:
URL: https://github.com/apache/arrow-rs/pull/4407#issuecomment-1613532324

   I think @viirya  should also have a chance to review / comment on this prior to merge.
   
   The justification as i understand it is that the primitive kernels are much faster anyways for this kind of operation and so including native dictionary creation is both slower as well as larger code and harder to work with


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org