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

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


The following commit(s) were added to refs/heads/branch-2.9 by this push:
     new 98c1629f9ae [fix][broker] Unregister topic policy listener if managed 
ledger close failed (#17652)
98c1629f9ae is described below

commit 98c1629f9ae5618ca166e9de47d8323a051ae36e
Author: Penghui Li <[email protected]>
AuthorDate: Thu Sep 15 10:35:44 2022 +0800

    [fix][broker] Unregister topic policy listener if managed ledger close 
failed (#17652)
    
    (cherry picked from commit dc5499794e0567ce87fc8ecdba5611a6d1fdbc26)
---
 .../org/apache/pulsar/broker/service/persistent/PersistentTopic.java     | 1 +
 1 file changed, 1 insertion(+)

diff --git 
a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java
 
b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java
index 99f12d1b1a9..3e15151b78a 100644
--- 
a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java
+++ 
b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java
@@ -1306,6 +1306,7 @@ public class PersistentTopic extends AbstractTopic
                 public void closeFailed(ManagedLedgerException exception, 
Object ctx) {
                     log.error("[{}] Failed to close managed ledger, proceeding 
anyway.", topic, exception);
                     brokerService.removeTopicFromCache(topic);
+                    unregisterTopicPolicyListener();
                     closeFuture.complete(null);
                 }
             }, null);

Reply via email to