maytasm commented on a change in pull request #9724:
URL: https://github.com/apache/druid/pull/9724#discussion_r412720994
##########
File path:
integration-tests/src/test/java/org/apache/druid/tests/coordinator/duty/ITAutoCompactionTest.java
##########
@@ -251,6 +251,9 @@ private void submitCompactionConfig(Integer
maxRowsPerSegment, Period skipOffset
null);
compactionResource.submitCompactionConfig(compactionConfig);
+ // Wait for compaction config to persist
+ Thread.sleep(2000);
Review comment:
I actually do make calls to config/compaction and
config/compaction/{datasource} to verify that the compaction is updated. But I
did see that the old config was apply to the compaction when the test runs in
the following sequence... post config 1 datasource A-> verify config 1 from the
two get calls -> post config 2 datasource A-> verify config 2 from the two get
calls -> force compaction datasource A
Will look at this more closely in a separate PR. Lets keep this hear to
reduce intermittent failure but I'll revisit later.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
