Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21794
BTW if it becomes necessary to not change the semantics, I think the
methods could at least be streamlined a bit:
```
if (x < y) {
-1
} else if (x > y) {
1
} else if (x == y) {
0
} else {
if (java.lang.Double.isNaN(x)) {
if (java.lang.Double.isNaN(y)) {
0
} else {
1
}
} else {
-1
}
}
```
More tests can't hurt, too.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
