[Native] Advance Velox

Author: aditi-pandit

presto-native-execution/velox

@@ -59,6 +59,7 @@
 import static com.facebook.presto.common.type.StandardTypes.ROW;
 import static com.facebook.presto.common.type.StandardTypes.SMALLINT;
 import static com.facebook.presto.common.type.StandardTypes.TDIGEST;
+import static com.facebook.presto.common.type.StandardTypes.TIME;
 import static com.facebook.presto.common.type.StandardTypes.TIMESTAMP;
 import static com.facebook.presto.common.type.StandardTypes.TIMESTAMP_WITH_TIME_ZONE;
 import static com.facebook.presto.common.type.StandardTypes.TINYINT;
@@ -84,6 +85,7 @@ public class NativeTypeManager
                     TINYINT,
                     BOOLEAN,
                     DATE,
+                    TIME,
                     INTEGER,
                     DOUBLE,
                     SMALLINT,

presto-native-sidecar-plugin/src/main/java/com/facebook/presto/sidecar/typemanager/NativeTypeManager.java

@@ -75,8 +75,8 @@ public void testApproximateCountDistinct()
                 approxDistinctUnsupportedSignatureError, true);
 
         // test time
-        assertQueryFails("SELECT approx_distinct(CAST(from_unixtime(custkey) AS TIME)) FROM orders", timeTypeUnsupportedError, true);
-        assertQueryFails("SELECT approx_distinct(CAST(from_unixtime(custkey) AS TIME), 0.023) FROM orders", timeTypeUnsupportedError, true);
+        assertQueryFails("SELECT approx_distinct(CAST(from_unixtime(custkey) AS TIME)) FROM orders", approxDistinctUnsupportedSignatureError, true);
+        assertQueryFails("SELECT approx_distinct(CAST(from_unixtime(custkey) AS TIME), 0.023) FROM orders", approxDistinctUnsupportedSignatureError, true);
 
         // test time with time zone
         assertQueryFails("SELECT approx_distinct(CAST(from_unixtime(custkey) AS TIME WITH TIME ZONE)) FROM orders", timeTypeUnsupportedError, true);

presto-native-tests/src/test/java/com/facebook/presto/nativetests/AbstractTestAggregationsNative.java

@@ -64,7 +64,7 @@ protected void createTables()
     }
 
     /// TIME datatype is not supported in Prestissimo. See issue: https://github.com/prestodb/presto/issues/18844.
-    @Override
+    /*@Override
     @Test
     public void testTimeLiterals()
     {
@@ -80,18 +80,18 @@ public void testTimeLiterals()
         assertQueryFails("SELECT TIME '3:04:05 +06:00'", timeTypeUnsupportedError);
         assertQueryFails("SELECT TIME '3:04:05 +0507'", timeTypeUnsupportedError);
         assertQueryFails("SELECT TIME '3:04:05 +03'", timeTypeUnsupportedError);
-    }
+    }*/
 
     /// TIME datatype is not supported in Prestissimo. See issue: https://github.com/prestodb/presto/issues/18844.
-    @Override
+    /*@Override
     @Test
     public void testLocallyUnrepresentableTimeLiterals()
     {
         LocalTime localTimeThatDidNotOccurOn20120401 = LocalTime.of(2, 10);
         checkState(ZoneId.systemDefault().getRules().getValidOffsets(localTimeThatDidNotOccurOn20120401.atDate(LocalDate.of(2012, 4, 1))).isEmpty(), "This test assumes certain JVM time zone");
         @Language("SQL") String sql = DateTimeFormatter.ofPattern("'SELECT TIME '''HH:mm:ss''").format(localTimeThatDidNotOccurOn20120401);
         assertQueryFails(sql, timeTypeUnsupportedError);
-    }
+    }*/
 
     // todo: turn on these test cases when the sql invoked functions are extracted into a plugin module.
     @Override