[GitHub] [druid] jihoonson commented on a change in pull request #12207: Lazy instantiation for segmentKillers, segmentMovers, and segmentArchivers

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

jihoonson commented on a change in pull request #12207:
URL: https://github.com/apache/druid/pull/12207#discussion_r800940091



##########
File path: indexing-service/src/test/java/org/apache/druid/indexing/common/task/batch/parallel/AbstractParallelIndexSupervisorTaskTest.java
##########
@@ -183,8 +183,8 @@
           null
       );
 
-  protected static final double DEFAULT_TRANSIENT_TASK_FAILURE_RATE = 0.3;
-  protected static final double DEFAULT_TRANSIENT_API_FAILURE_RATE = 0.3;
+  protected static final double DEFAULT_TRANSIENT_TASK_FAILURE_RATE = 0.2;

Review comment:
       Some tests failed because of timeout. As those tests pass with no issue in my computer, one reason that caused the timeout could be too often retries. Reducing failure rate can mitigate too often retry problem.




-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

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