BewareMyPower commented on issue #16813:
URL: https://github.com/apache/pulsar/issues/16813#issuecomment-1198241372

   This behavior is caused by the behavior change of 
`ManagedLedgerImpl#rollCurrentLedgerIfFull` in #14622, which should only be 
included in branch-2.11.
   
   Then I checked the test in branch-2.9 and branch-2.10.
   
   For branch-2.9, when 53615e05c cherry-picked #15837, it also changed the 
test to
   
   ```java
           Awaitility.await().untilAsserted(() -> {
               assertEquals(managedLedger.getLedgersInfo().size(), 3);
               assertEquals(managedLedger.getState(), 
ManagedLedgerImpl.State.LedgerOpened);
           })
   ```
   
   see 
[here](https://github.com/apache/pulsar/commit/53615e05cff4010ef0834f15f057b78c1eb6fc77#diff-453ce631d538108f712bf39c96af781ea44a87875a9c70cc806ad7cf3a6af518)
   
   For branch-2.10, this test failure was fixed by 
https://github.com/apache/pulsar/pull/15995.
   
   I will push a test to fix the test in branch-2.8 soon. /cc @mattisonchao 
@codelipenghui @nicoloboschi 
   
   Regarding the master branch and the incoming 2.11.0 release, 
https://github.com/apache/pulsar/pull/16806, which is marked as the release 
blocker, should restore the original behavior.


-- 
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]

Reply via email to