maropu commented on issue #26654: [SPARK-30009][CORE][SQL] Support different 
floating-point Ordering for Scala 2.12 / 2.13
URL: https://github.com/apache/spark/pull/26654#issuecomment-559625360
 
 
   Yea, that should be consisntent. `Nan` is the maximum value among 
double/float values including infinity in terms of SQL. So, the result above is 
correct and that's the same with the Pg one;
   ```
   postgres=# select * from t;
        v     
   -----------
    -Infinity
     Infinity
       1e-307
       1e+308
          NaN
   (5 rows)
   
   postgres=# select v < 'NaN' from t;
    ?column? 
   ----------
    t
    t
    t
    t
    f
   (5 rows)
   ```
   Anyway, the fix in this pr looks pretty reasonable to me.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to