Presto Aggregate Max(Nan) yields non Nan value

[kgpai]

Bug description

Currently presto aggregate function max seems to ignore NaN . As can be seen from the queries below.

c++ ======================================
presto:feed> select max(x) from (values 4.0,nan(),null) T(x);
   4.0

presto:feed> select max(x) from (values nan(),nan(),null) T(x);
 -1.7976931348623157E308

presto:feed> select min(x) from (values 4.0,nan(),null) T(x);
   NaN

presto:feed> select min(x) from (values nan(),nan(),null) T(x);
   NaN


java =====================================
presto:feed> select max(x) from (values 4.0,nan(),null) T(x);
   4.0

presto:feed> select max(x) from (values nan(),nan(),null) T(x);
   NaN

presto:feed> select min(x) from (values 4.0,nan(),null) T(x);
   4.0

presto:feed> select min(x) from (values nan(),nan(),null) T(x);
   NaN

System information

N/A

Relevant logs

No response

[mbasmanova]

Notice that in Java, for some reason, the order matters:

presto:di> select max(x) from (values 4.0,nan(),null) T(x);;
 _col0
-------
   4.0
(1 row)

presto:di> select max(x) from (values nan(),4.0,null) T(x);
 _col0
-------
   NaN
(1 row)

This might be a bug in Java. Worth creating an issue in prestodb to ask.

[kgpai]

Created issue here : prestodb/presto#21877 .