[fix][broker] managedLedger.getConfig().getProperties().putAll(proper…

…ties) NPE (apache#20361) (apache#5932) (cherry picked from commit aa7decc) Co-authored-by: Heesung Sohn <103456639+heesung-sn@users.noreply.github.com>
liangyepianzhou · Oct 19, 2023 · 5f6ed1b · 5f6ed1b
1 parent c84a829
commit 5f6ed1b
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java
@@ -697,7 +697,11 @@ private CompletableFuture<Void> internalUpdateNonPartitionedTopicProperties(Map<
 
                     @Override
                     public void updatePropertiesComplete(Map<String, String> properties, Object ctx) {
+                        if (managedLedger.getConfig().getProperties() == null) {
+                            managedLedger.getConfig().setProperties(new HashMap<>());
+                        }
                         managedLedger.getConfig().getProperties().putAll(properties);
+
                         future.complete(null);
                     }