marekczajkowski commented on issue #23161: URL: https://github.com/apache/pulsar/issues/23161#issuecomment-2307152038
There is a race condition in this test CurrentLedgerRolloverIfFullTest#testCurrentLedgerRolloverIfFull Please find attached screenshot. As you can see there is maxEntriesPerLedger=2 and than 10 messages are sent. That would mean we have 5 ledgers each with 2 messages. Then a new ledger is being created to accept next upcoming messages but in many cases it does not happen until assertion is done which compares ledgers to 5 see commented out code. I believe it should wait for this new ledger and than compare the correct number of 6 ledgers <img width="825" alt="Zrzut ekranu 2024-08-23 o 15 43 05" src="https://github.com/user-attachments/assets/7e3973ac-c221-4f34-b31d-9277e9900e59";> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
