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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/12/08 20:42:02 UTC

[GitHub] [arrow] Dandandan commented on pull request #8685: ARROW-10216: [Rust] Simd implementation for primitive min/max kernels

Dandandan commented on pull request #8685:
URL: https://github.com/apache/arrow/pull/8685#issuecomment-740997894


   @jhorstmann 
   I wonder if we should support a standardized ordering like  IEEE 754 instead?
   
   There is a (unstable) implementation in rust std for it.
   https://doc.rust-lang.org/beta/std/primitive.f64.html#method.total_cmp
   
   Sounds something we could have in the source for now (it is only a few lines), I think it might also simplify some code like the current sort kernels? What do you think?


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

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