diff --git a/presto-main/src/main/java/com/facebook/presto/memory/NodeMemoryConfig.java b/presto-main/src/main/java/com/facebook/presto/memory/NodeMemoryConfig.java index 22190441c2b0d..33bf4f175b4a4 100644 --- a/presto-main/src/main/java/com/facebook/presto/memory/NodeMemoryConfig.java +++ b/presto-main/src/main/java/com/facebook/presto/memory/NodeMemoryConfig.java @@ -42,7 +42,7 @@ public class NodeMemoryConfig private DataSize softMaxQueryTotalMemoryPerNode; private DataSize heapHeadroom = new DataSize(AVAILABLE_HEAP_MEMORY * 0.3, BYTE); - private boolean verboseExceededMemoryLimitErrorsEnabled; + private boolean verboseExceededMemoryLimitErrorsEnabled = true; @NotNull public DataSize getMaxQueryBroadcastMemory() diff --git a/presto-main/src/test/java/com/facebook/presto/memory/TestNodeMemoryConfig.java b/presto-main/src/test/java/com/facebook/presto/memory/TestNodeMemoryConfig.java index 6c1524a6287a8..abe62103f653c 100644 --- a/presto-main/src/test/java/com/facebook/presto/memory/TestNodeMemoryConfig.java +++ b/presto-main/src/test/java/com/facebook/presto/memory/TestNodeMemoryConfig.java @@ -40,7 +40,7 @@ public void testDefaults() .setSoftMaxQueryTotalMemoryPerNode(new DataSize(AVAILABLE_HEAP_MEMORY * 0.3, BYTE)) .setHeapHeadroom(new DataSize(AVAILABLE_HEAP_MEMORY * 0.3, BYTE)) .setReservedPoolEnabled(true) - .setVerboseExceededMemoryLimitErrorsEnabled(false)); + .setVerboseExceededMemoryLimitErrorsEnabled(true)); } @Test @@ -54,7 +54,7 @@ public void testExplicitPropertyMappings() .put("query.soft-max-total-memory-per-node", "2GB") .put("memory.heap-headroom-per-node", "1GB") .put("experimental.reserved-pool-enabled", "false") - .put("memory.verbose-exceeded-memory-limit-errors-enabled", "true") + .put("memory.verbose-exceeded-memory-limit-errors-enabled", "false") .build(); NodeMemoryConfig expected = new NodeMemoryConfig() @@ -65,7 +65,7 @@ public void testExplicitPropertyMappings() .setSoftMaxQueryTotalMemoryPerNode(new DataSize(2, GIGABYTE)) .setHeapHeadroom(new DataSize(1, GIGABYTE)) .setReservedPoolEnabled(false) - .setVerboseExceededMemoryLimitErrorsEnabled(true); + .setVerboseExceededMemoryLimitErrorsEnabled(false); assertFullMapping(properties, expected); } @@ -81,7 +81,7 @@ public void testOutOfRangeBroadcastMemoryLimit() .put("query.soft-max-total-memory-per-node", "2GB") .put("memory.heap-headroom-per-node", "1GB") .put("experimental.reserved-pool-enabled", "false") - .put("memory.verbose-exceeded-memory-limit-errors-enabled", "true") + .put("memory.verbose-exceeded-memory-limit-errors-enabled", "false") .build(); NodeMemoryConfig expected = new NodeMemoryConfig() @@ -92,7 +92,7 @@ public void testOutOfRangeBroadcastMemoryLimit() .setSoftMaxQueryTotalMemoryPerNode(new DataSize(2, GIGABYTE)) .setHeapHeadroom(new DataSize(1, GIGABYTE)) .setReservedPoolEnabled(false) - .setVerboseExceededMemoryLimitErrorsEnabled(true); + .setVerboseExceededMemoryLimitErrorsEnabled(false); assertFullMapping(properties, expected); }