[ArnavBalyan]

Purpose

• Float and double comparators reused the integer ternary, which is not correct since nan compares equal to everything and signed zero is treated as equal.
• This silently misorders any sort or clustering on a float or double column containing nan, and can cause buggy java sort.
• Switch the float and double cases to the jdk total order comparison.

Tests

• UT

[ArnavBalyan]

cc @leaves12138 @JingsongLi thanks! :)

[JingsongLi]

+1