Skip to content

Commit 8d9d497

Browse files
authored
[fix][test] Fix flaky ManagedLedgerTest.testNoRetention (#25467)
1 parent 05a703b commit 8d9d497

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2309,8 +2309,10 @@ public void testNoRetention() throws Exception {
23092309
c1 = ml.openCursor("c1noretention");
23102310
ml.addEntry("shortmessage".getBytes());
23112311
c1.skipEntries(1, IndividualDeletedEntries.Exclude);
2312-
// sleep for trim
2313-
Thread.sleep(1000);
2312+
// Explicitly trigger trimming and wait for it to complete
2313+
CompletableFuture<Void> trimFuture = new CompletableFuture<>();
2314+
ml.trimConsumedLedgersInBackground(trimFuture);
2315+
trimFuture.join();
23142316
ml.close();
23152317

23162318
assertTrue(ml.getLedgersInfoAsList().size() <= 1);

0 commit comments

Comments
 (0)