From 9b020152c668a4d5a083c2e58c464c506858da2f Mon Sep 17 00:00:00 2001 From: gaoran10 Date: Thu, 8 Jul 2021 03:13:37 +0800 Subject: [PATCH] increase the maximum rollover time to 10 seconds in test `testLedgerReachMaximumRolloverTime` --- .../bookkeeper/mledger/impl/ManagedLedgerTest.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java b/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java index a150f0f0dfb1f..922af4374fdcf 100644 --- a/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java +++ b/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java @@ -2877,16 +2877,16 @@ public void testOpEntryAdd_toString_doesNotThrowNPE(){ public void testLedgerReachMaximumRolloverTime() throws Exception { ManagedLedgerConfig config = new ManagedLedgerConfig(); config.setMinimumRolloverTime(1, TimeUnit.MILLISECONDS); - config.setMaximumRolloverTime(1, TimeUnit.SECONDS); + config.setMaximumRolloverTime(10, TimeUnit.SECONDS); ManagedLedger ml = factory.open("ledger-reach-maximum-rollover-time", config); long firstLedgerId = ((PositionImpl) ml.addEntry("test".getBytes())).getLedgerId(); - // the ledger rollover scheduled time is between 1000 and 1050 ms, - // wait 1100 ms, the ledger should be rolled over. + // the ledger rollover scheduled time is between 1000 * 10 and 1000 * 10 + 500 ms, + // wait 1000 * 12 ms, the ledger should be rolled over. Awaitility.await() - .atMost(1100, TimeUnit.MILLISECONDS) - .pollInterval(100, TimeUnit.MILLISECONDS) + .atMost(12, TimeUnit.SECONDS) + .pollInterval(500, TimeUnit.MILLISECONDS) .until(() -> firstLedgerId != ((PositionImpl) ml.addEntry("test".getBytes())).getLedgerId()); }