[pulsar] branch branch-2.11 updated: [fix][broker] Unregister topic policy listener if managed ledger close failed (#17652)

[te...@apache.org]

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

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


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

commit 91fbfdafe206517eb52ebcc269dcbf110dc7ea6c
Author: Penghui Li <pe...@apache.org>
AuthorDate: Thu Sep 15 10:35:44 2022 +0800

    [fix][broker] Unregister topic policy listener if managed ledger close failed (#17652)
---
 .../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 909086c1373..60b08413235 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
@@ -1313,6 +1313,7 @@ public class PersistentTopic extends AbstractTopic implements Topic, AddEntryCal
                 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);