Fix round for float value when input out of long range

Author: hantangwangd

presto-main/src/main/java/com/facebook/presto/operator/scalar/MathFunctions.java

@@ -1136,7 +1136,7 @@ public static long roundFloat(@SqlType(StandardTypes.REAL) long num, @SqlType(St
         catch (ArithmeticException e) {
             // Use BigDecimal if the value is out of the range of long.
             BigDecimal bigDecimal = new BigDecimal(numInFloat);
-            return floatToRawIntBits(bigDecimal.setScale((int) decimals, HALF_UP).longValue());
+            return floatToRawIntBits(bigDecimal.setScale((int) decimals, HALF_UP).floatValue());
         }
     }
 

presto-main/src/test/java/com/facebook/presto/operator/scalar/TestMathFunctions.java

@@ -709,6 +709,19 @@ public void testSecureRandom()
         assertInvalidFunction("secure_random(DECIMAL '5.0', DECIMAL '-5.0')", "upper bound must be greater than lower bound");
     }
 
+    @Test
+    public void testRoundForUnderlyingValueOutOfRange()
+    {
+        // Round data of `REAL` type with underlying value out of long range should work well.
+        // See issue https://github.com/prestodb/presto/issues/23763
+        assertFunction("round(REAL '1.0E19', 1)", REAL, 1.0E19f);
+        assertFunction("round(REAL '1.0E19', 10)", REAL, 1.0E19f);
+        assertFunction("round(REAL '1.0E19', 100)", REAL, 1.0E19f);
+        assertFunction("round(REAL '9999999999999999999.9', 1)", REAL, 9999999999999999999.9f);
+        assertFunction("round(REAL '9999999999999999999.99', 10)", REAL, 9999999999999999999.99f);
+        assertFunction("round(REAL '9999999999999999999.999', 100)", REAL, 9999999999999999999.999f);
+    }
+
     @Test
     public void testRound()
     {