bookkeeper-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From si...@apache.org
Subject bookkeeper git commit: BOOKKEEPER-946: Provide an option to delay auto recovery of lost bookies
Date Sat, 17 Dec 2016 01:44:29 GMT
Repository: bookkeeper
Updated Branches:
  refs/heads/master be99f024d -> 669ab4ac3


BOOKKEEPER-946: Provide an option to delay auto recovery of lost bookies

Fixing a bug in the test AuditorLedgerCheckerTest.testDelayedAuditOfLostBookies which
fails sometimes with:
AuditorLedgerCheckerTest.testDelayedAuditOfLostBookies:367->_testDelayedAuditOfLostBookies:345
audit of lost bookie isn't delayed

Author: Rithin <rithin.shetty@salesforce.com>

Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>

Closes #82 from rithin-shetty/audit_delay_fix


Project: http://git-wip-us.apache.org/repos/asf/bookkeeper/repo
Commit: http://git-wip-us.apache.org/repos/asf/bookkeeper/commit/669ab4ac
Tree: http://git-wip-us.apache.org/repos/asf/bookkeeper/tree/669ab4ac
Diff: http://git-wip-us.apache.org/repos/asf/bookkeeper/diff/669ab4ac

Branch: refs/heads/master
Commit: 669ab4ac32bcbf6b3d883a07ed942d36d25b8a6e
Parents: be99f02
Author: Rithin <rithin.shetty@salesforce.com>
Authored: Fri Dec 16 17:44:24 2016 -0800
Committer: Sijie Guo <sijie@apache.org>
Committed: Fri Dec 16 17:44:24 2016 -0800

----------------------------------------------------------------------
 .../apache/bookkeeper/replication/AuditorLedgerCheckerTest.java   | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/bookkeeper/blob/669ab4ac/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorLedgerCheckerTest.java
----------------------------------------------------------------------
diff --git a/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorLedgerCheckerTest.java
b/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorLedgerCheckerTest.java
index f0a4fed..974761e 100644
--- a/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorLedgerCheckerTest.java
+++ b/bookkeeper-server/src/test/java/org/apache/bookkeeper/replication/AuditorLedgerCheckerTest.java
@@ -364,6 +364,9 @@ public class AuditorLedgerCheckerTest extends MultiLedgerManagerTestCase
{
      */
     @Test(timeout=60000)
     public void testDelayedAuditOfLostBookies() throws Exception {
+        // wait for a second so that the initial periodic check finishes
+        Thread.sleep(1000);
+
         _testDelayedAuditOfLostBookies();
     }
 


Mime
View raw message