From 8c5875def79998c1197a3806f0829d73576de936 Mon Sep 17 00:00:00 2001 From: Matteo Merli Date: Thu, 22 Sep 2016 11:48:19 -0700 Subject: [PATCH] In managed ledger BK test, wait for background ledger roll-over to complete before shutting down Fixes #16: Intermittent test failure in ManagedLedgerBkTest.testConcurrentMarkDelete --- .../apache/bookkeeper/mledger/impl/ManagedLedgerBkTest.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerBkTest.java b/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerBkTest.java index 03f51a2067720..e093385ced9bb 100644 --- a/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerBkTest.java +++ b/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerBkTest.java @@ -273,6 +273,9 @@ public void testConcurrentMarkDelete() throws Exception { future.get(); } + // Since in this test we roll-over the cursor ledger every 10 entries acknowledged, the background roll back + // might still be happening when the futures are completed. + Thread.sleep(1000); factory.shutdown(); }