Enable timeout by default in Alluxio

prestodb · Apr 21, 2021 · 7c8b6f5 · 7c8b6f5
1 parent 3e850ba
commit 7c8b6f5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/presto-cache/src/main/java/com/facebook/presto/cache/alluxio/AlluxioCacheConfig.java b/presto-cache/src/main/java/com/facebook/presto/cache/alluxio/AlluxioCacheConfig.java
@@ -29,7 +29,7 @@ public class AlluxioCacheConfig
     private String metricsDomain = "com.facebook.alluxio";
     private DataSize maxCacheSize = new DataSize(2, GIGABYTE);
     private boolean configValidationEnabled;
-    private boolean timeoutEnabled;
+    private boolean timeoutEnabled = true;
     private boolean cacheQuotaEnabled;
     private Duration timeoutDuration = new Duration(60, SECONDS);
     private int timeoutThreads = 64;

diff --git a/presto-cache/src/test/java/com/facebook/presto/cache/alluxio/TestAlluxioCacheConfig.java b/presto-cache/src/test/java/com/facebook/presto/cache/alluxio/TestAlluxioCacheConfig.java
@@ -42,7 +42,7 @@ public void testDefaults()
                 .setMetricsCollectionEnabled(true)
                 .setMetricsDomain("com.facebook.alluxio")
                 .setTimeoutDuration(new Duration(60, SECONDS))
-                .setTimeoutEnabled(false)
+                .setTimeoutEnabled(true)
                 .setTimeoutThreads(64)
                 .setCacheQuotaEnabled(false));
     }
@@ -60,7 +60,7 @@ public void testExplicitPropertyMappings()
                 .put("cache.alluxio.metrics-domain", "test.alluxio")
                 .put("cache.alluxio.metrics-enabled", "false")
                 .put("cache.alluxio.timeout-duration", "120s")
-                .put("cache.alluxio.timeout-enabled", "true")
+                .put("cache.alluxio.timeout-enabled", "false")
                 .put("cache.alluxio.timeout-threads", "512")
                 .put("cache.alluxio.quota-enabled", "true")
                 .build();
@@ -75,7 +75,7 @@ public void testExplicitPropertyMappings()
                 .setJmxClass("test.TestJmxSink")
                 .setConfigValidationEnabled(true)
                 .setTimeoutDuration(new Duration(120, SECONDS))
-                .setTimeoutEnabled(true)
+                .setTimeoutEnabled(false)
                 .setTimeoutThreads(512)
                 .setCacheQuotaEnabled(true);