-
Notifications
You must be signed in to change notification settings - Fork 1k
Closed
Labels
arrowChanges to the arrow crateChanges to the arrow crateenhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changeloghelp wanted
Description
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
We have gradually converged on using total ordering everywhere, as opposed to special casing NaN for floating point numbers. It would appear that the min and max kernels have not yet been updated.
Describe the solution you'd like
We should update the min and max kernels to use total ordering.
Describe alternatives you've considered
Additional context
apache/datafusion#8031 is an upstream report of this issue
#5032 may be related to making the aggregate kernels use total ordering
Metadata
Metadata
Assignees
Labels
arrowChanges to the arrow crateChanges to the arrow crateenhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changeloghelp wanted