add test for testDefaults

sopel39 · Dec 15, 2018 · e60baed · e60baed
1 parent f5df4c9
commit e60baed
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/presto-main/src/test/java/com/facebook/presto/execution/TestQueryManagerConfig.java b/presto-main/src/test/java/com/facebook/presto/execution/TestQueryManagerConfig.java
@@ -48,7 +48,8 @@ public void testDefaults()
                 .setInitializationRequiredWorkers(1)
                 .setInitializationTimeout(new Duration(5, TimeUnit.MINUTES))
                 .setRequiredWorkers(1)
-                .setRequiredWorkersMaxWait(new Duration(5, TimeUnit.MINUTES)));
+                .setRequiredWorkersMaxWait(new Duration(5, TimeUnit.MINUTES))
+                .setWaitForQueryStatsAfterFailed(false));
     }
 
     @Test
@@ -77,7 +78,7 @@ public void testExplicitPropertyMappings()
                 .put("query-manager.initialization-timeout", "1m")
                 .put("query-manager.required-workers", "333")
                 .put("query-manager.required-workers-max-wait", "33m")
-                .put("query.wait-stats-after-failed-enabled", "false")
+                .put("query.wait-stats-after-failed-enabled", "true")
                 .build();
 
         QueryManagerConfig expected = new QueryManagerConfig()
@@ -103,7 +104,7 @@ public void testExplicitPropertyMappings()
                 .setInitializationTimeout(new Duration(1, TimeUnit.MINUTES))
                 .setRequiredWorkers(333)
                 .setRequiredWorkersMaxWait(new Duration(33, TimeUnit.MINUTES))
-                .setWaitForQueryStatsAfterFailed(false);
+                .setWaitForQueryStatsAfterFailed(true);
 
         ConfigAssertions.assertFullMapping(properties, expected);
     }