[GitHub] [druid] maytasm commented on a change in pull request #8987: Adding support for autoscaling in GCE

[GitBox <gi...@apache.org>]

maytasm commented on a change in pull request #8987:
URL: https://github.com/apache/druid/pull/8987#discussion_r412557178



##########
File path: integration-tests/src/test/java/org/apache/druid/tests/coordinator/duty/ITAutoCompactionTest.java
##########
@@ -237,7 +237,7 @@ private void verifyQuery(String queryResource) throws Exception
         fullDatasourceName
     );
 
-    queryHelper.testQueriesFromString(queryResponseTemplate, 2);
+    queryHelper.testQueriesFromString(queryResponseTemplate, 5);

Review comment:
       I don't think this will help. The 'timesToRun' number requires ALL of the running to pass. If any one of the running failed then the test will fail. Increasing timesToRun will make the test slower but does not help with retrying intermittent failure. I have a PR to fix this issue at https://github.com/apache/druid/pull/9739 
   Sorry for the trouble from the intermittent failure.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org