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/14 01:08:10 UTC
[pulsar] branch master updated: [fix][broker] Fix SystemTopicBasedTopicPoliciesService NPE issue (#17602)
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 b789b08c8c5 [fix][broker] Fix SystemTopicBasedTopicPoliciesService NPE issue (#17602)
b789b08c8c5 is described below
commit b789b08c8c573aa12127a131ddd9b2de9dc48eec
Author: Jiwei Guo <te...@apache.org>
AuthorDate: Wed Sep 14 09:08:02 2022 +0800
[fix][broker] Fix SystemTopicBasedTopicPoliciesService NPE issue (#17602)
---
.../pulsar/broker/service/SystemTopicBasedTopicPoliciesService.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/SystemTopicBasedTopicPoliciesService.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/SystemTopicBasedTopicPoliciesService.java
index 0e0e4950c36..f5db6e2311b 100644
--- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/SystemTopicBasedTopicPoliciesService.java
+++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/SystemTopicBasedTopicPoliciesService.java
@@ -246,7 +246,6 @@ public class SystemTopicBasedTopicPoliciesService implements TopicPoliciesServic
ownedBundlesCountPerNamespace.get(namespace).incrementAndGet();
result.complete(null);
} else {
- ownedBundlesCountPerNamespace.putIfAbsent(namespace, new AtomicInteger(1));
prepareInitPoliciesCache(namespace, result);
}
}
@@ -258,6 +257,7 @@ public class SystemTopicBasedTopicPoliciesService implements TopicPoliciesServic
CompletableFuture<SystemTopicClient.Reader<PulsarEvent>> readerCompletableFuture =
createSystemTopicClientWithRetry(namespace);
readerCaches.put(namespace, readerCompletableFuture);
+ ownedBundlesCountPerNamespace.putIfAbsent(namespace, new AtomicInteger(1));
readerCompletableFuture.thenAccept(reader -> {
initPolicesCache(reader, result);
result.thenRun(() -> readMorePolicies(reader));