This is an automated email from the ASF dual-hosted git repository.

zixuan 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 d2d05c2fc5d [fix][test] Fix ManagedCursorTest.testForceCursorRecovery 
(#23518)
d2d05c2fc5d is described below

commit d2d05c2fc5dd0d89be63fea1fdcd927726ff85e5
Author: CrazyCoder <[email protected]>
AuthorDate: Tue Nov 5 18:16:55 2024 +0800

    [fix][test] Fix ManagedCursorTest.testForceCursorRecovery (#23518)
---
 .../java/org/apache/bookkeeper/mledger/impl/ManagedCursorTest.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedCursorTest.java
 
b/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedCursorTest.java
index 587f87a7d1d..b4ab673facb 100644
--- 
a/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedCursorTest.java
+++ 
b/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedCursorTest.java
@@ -4828,7 +4828,6 @@ public class ManagedCursorTest extends 
MockedBookKeeperTestCase {
 
     @Test
     void testForceCursorRecovery() throws Exception {
-        ManagedLedgerFactoryConfig managedLedgerFactoryConfig = new 
ManagedLedgerFactoryConfig();
         TestPulsarMockBookKeeper bk = new TestPulsarMockBookKeeper(executor);
         factory = new ManagedLedgerFactoryImpl(metadataStore, bk);
         ManagedLedgerConfig config = new ManagedLedgerConfig();
@@ -4874,8 +4873,9 @@ public class ManagedCursorTest extends 
MockedBookKeeperTestCase {
                 final OpenCallback cb, final Object ctx) {
             if (ledgerErrors.containsKey(lId)) {
                 cb.openComplete(ledgerErrors.get(lId), null, ctx);
+            } else {
+                super.asyncOpenLedger(lId, digestType, passwd, cb, ctx);
             }
-            super.asyncOpenLedger(lId, digestType, passwd, cb, ctx);
         }
     }
 

Reply via email to