You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by te...@apache.org on 2022/09/15 02:35:52 UTC
[pulsar] branch master updated: [fix][broker] Unregister topic policy listener if managed ledger close failed (#17652)
This is an automated email from the ASF dual-hosted git repository.
technoboy 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 dc5499794e0 [fix][broker] Unregister topic policy listener if managed ledger close failed (#17652)
dc5499794e0 is described below
commit dc5499794e0567ce87fc8ecdba5611a6d1fdbc26
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 4d48f9c627e..52bb165a61e 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
@@ -1323,6 +1323,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);