eolivelli commented on a change in pull request #2166: Enhance
deferLedgerLockReleaseOfFailedLedger in ReplicationWorker
URL: https://github.com/apache/bookkeeper/pull/2166#discussion_r329088328
##########
File path:
bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/TestReplicationWorker.java
##########
@@ -502,10 +506,11 @@ public void
testBookiesNotAvailableScenarioForReplicationWorker() throws Excepti
@Override
public void run() {
try {
- Thread.sleep(4000);
+ Thread.sleep(3000);
isBookieRestarted.set(true);
+ Thread.sleep(500);
/*
- * after sleeping for 4000 msecs, restart one of the
+ * after sleeping for 3000 msecs, restart one of the
Review comment:
do we have a better to wait ?
Just I don't want to see another flaky test on CI
not blocker, the test is already using Thraed.sleep
----------------------------------------------------------------
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]
With regards,
Apache Git Services
