This is an automated email from the ASF dual-hosted git repository.
lhotari pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 20777df8ea8 [fix][test] Fix flaky testRetentionSize in
ManagedLedgerTest (#25423)
20777df8ea8 is described below
commit 20777df8ea83571af92c9ee3ab9e228dcbf9cbd6
Author: Matteo Merli <[email protected]>
AuthorDate: Mon Mar 30 15:51:17 2026 -0700
[fix][test] Fix flaky testRetentionSize in ManagedLedgerTest (#25423)
---
.../java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java | 5 +++--
1 file changed, 3 insertions(+), 2 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 ac292e1fe15..b1066c297c4 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
@@ -2490,8 +2490,9 @@ public class ManagedLedgerTest extends
MockedBookKeeperTestCase {
for (int i = 0; i < totalMessage; i++) {
position = ml.addEntry(message);
}
- // all ledgers are not delete yet since no entry has been acked for c1
- assertEquals(ml.getLedgersInfoAsList().size(), totalMessage);
+ // all ledgers are not deleted yet since no entry has been acked for c1
+ // Use >= because the current (empty) ledger may or may not have been
created yet
+ assertTrue(ml.getLedgersInfoAsList().size() >= totalMessage);
List<Entry> entryList = c1.readEntries(totalMessage);
if (null != position) {