sunchao commented on issue #2568:
URL: https://github.com/apache/arrow-rs/issues/2568#issuecomment-1230656665

   > I think total_cmp looks promising as it treats NaN ordering the same way 
as Spark/Postgresql. cc @sunchao to take a look too in case if I miss any point 
here. 😄
   
   I took a look too, and yes it looks like `total_cmp` exhibits the same 
behavior as Spark/Postgres/Snowflake etc, so I think it could be good idea to 
replace the existing code with it.


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