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

penghui pushed a commit to branch branch-2.8
in repository https://gitbox.apache.org/repos/asf/pulsar.git

commit 4829f79fbaf2940d220e0906091390cfcf0cad36
Author: Xiaoyu Hou <[email protected]>
AuthorDate: Wed Apr 6 20:49:01 2022 +0800

    [fix][broker] Return if reset in progress (#14978)
    
    ### Motivation
    
    - Fix bug. If cursor reset in progress, the callback method will call 
`resetFailed` but without return. This will cause the callback be invoked again
    
    ### Modifications
    
    - Just add `return` after callback invoke `resetFailed`  If cursor reset in 
progress
    
    (cherry picked from commit 81da8d3cd199fd6c1e4510a1c1c2ac71418efd5e)
---
 .../main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java  | 1 +
 1 file changed, 1 insertion(+)

diff --git 
a/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java
 
b/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java
index d03351f2da2..dfd4d6f220a 100644
--- 
a/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java
+++ 
b/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java
@@ -1045,6 +1045,7 @@ public class ManagedCursorImpl implements ManagedCursor {
                 resetCursorCallback.resetFailed(
                         new 
ManagedLedgerException.ConcurrentFindCursorPositionException("reset already in 
progress"),
                         position);
+                return;
             }
         }
 

Reply via email to