jhorstmann commented on pull request #8685: URL: https://github.com/apache/arrow/pull/8685#issuecomment-741606137
@Dandandan that sounds like good idea as it is a bit more standardized. The current ordering was choosen initially for the sort kernels, as it is the same ordering that Postgres implements. The main difference seems to be the handling of negative NaN. If we decide on a different ordering we should keep sort and min/max consistent. One nice thing about the current impl for min/max is that all numeric types are handled the same, with the compiler optimizing away the `is_nan` function for integer types. ---------------------------------------------------------------- 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: [email protected]
