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

Posted to github@arrow.apache.org by "Jimexist (via GitHub)" <gi...@apache.org> on 2023/06/03 15:16:05 UTC

[GitHub] [arrow-rs] Jimexist commented on pull request #4355: add min and max kernel

Jimexist commented on PR #4355:
URL: https://github.com/apache/arrow-rs/pull/4355#issuecomment-1575013974

   > These kernels already exist in arrow-arithmetic I think?
   > 
   > Also FYI I'm working on a rework of the way we represent scalars in arrow-rs - see #1047
   > 
   > Edit: i see now the difference, I think this should be implemented in DF as a comparison kernel followed by a selection kernel. This will help keep code bloat down
   > 
   > https://docs.rs/arrow-select/latest/arrow_select/zip/fn.zip.html should do what you want
   
   Thanks for the advice I have already done that in https://github.com/apache/arrow-datafusion/pull/6527/files


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