[native] Disable plan validation failure for timestamp with timezone …

…and ipaddress types for native execution.
prestodb · Nov 23, 2024 · 174f1ce · 174f1ce
1 parent 0c5fe5a
commit 174f1ce
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/presto-main/src/main/java/com/facebook/presto/sql/analyzer/FeaturesConfig.java b/presto-main/src/main/java/com/facebook/presto/sql/analyzer/FeaturesConfig.java
@@ -234,8 +234,8 @@ public class FeaturesConfig
     private boolean pushRemoteExchangeThroughGroupId;
     private boolean isOptimizeMultipleApproxPercentileOnSameFieldEnabled = true;
     private boolean nativeExecutionEnabled;
-    private boolean disableTimeStampWithTimeZoneForNative = true;
-    private boolean disableIPAddressForNative = true;
+    private boolean disableTimeStampWithTimeZoneForNative;
+    private boolean disableIPAddressForNative;
     private String nativeExecutionExecutablePath = "./presto_server";
     private String nativeExecutionProgramArguments = "";
     private boolean nativeExecutionProcessReuseEnabled = true;

diff --git a/presto-main/src/test/java/com/facebook/presto/sql/analyzer/TestFeaturesConfig.java b/presto-main/src/test/java/com/facebook/presto/sql/analyzer/TestFeaturesConfig.java
@@ -199,8 +199,8 @@ public void testDefaults()
                 .setPushRemoteExchangeThroughGroupId(false)
                 .setOptimizeMultipleApproxPercentileOnSameFieldEnabled(true)
                 .setNativeExecutionEnabled(false)
-                .setDisableTimeStampWithTimeZoneForNative(true)
-                .setDisableIPAddressForNative(true)
+                .setDisableTimeStampWithTimeZoneForNative(false)
+                .setDisableIPAddressForNative(false)
                 .setNativeExecutionExecutablePath("./presto_server")
                 .setNativeExecutionProgramArguments("")
                 .setNativeExecutionProcessReuseEnabled(true)