You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by si...@apache.org on 2016/12/17 01:44:29 UTC
bookkeeper git commit: BOOKKEEPER-946: Provide an option to delay
auto recovery of lost bookies
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 <ri...@salesforce.com>
Reviewers: Enrico Olivelli <eo...@gmail.com>, Sijie Guo <si...@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 <ri...@salesforce.com>
Authored: Fri Dec 16 17:44:24 2016 -0800
Committer: Sijie Guo <si...@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();
}