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/03/27 12:56:10 UTC
[pulsar] branch master updated: [improve][broker] Add log when update namespace policies with error. (#14850)
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 195b13c [improve][broker] Add log when update namespace policies with error. (#14850)
195b13c is described below
commit 195b13c427fb480d14f9813c534b9ba4c89dd671
Author: Jiwei Guo <te...@apache.org>
AuthorDate: Sun Mar 27 20:54:33 2022 +0800
[improve][broker] Add log when update namespace policies with error. (#14850)
---
.../apache/pulsar/broker/service/nonpersistent/NonPersistentTopic.java | 3 +++
.../org/apache/pulsar/broker/service/persistent/PersistentTopic.java | 3 +++
2 files changed, 6 insertions(+)
diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/nonpersistent/NonPersistentTopic.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/nonpersistent/NonPersistentTopic.java
index 4d86666..337b4c7 100644
--- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/nonpersistent/NonPersistentTopic.java
+++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/nonpersistent/NonPersistentTopic.java
@@ -1030,6 +1030,9 @@ public class NonPersistentTopic extends AbstractTopic implements Topic, TopicPol
return FutureUtil.waitForAll(consumerCheckFutures)
.thenCompose((___) -> checkReplicationAndRetryOnFailure());
+ }).exceptionally(ex -> {
+ log.error("[{}] update namespace polices : {} error", this.getName(), data, ex);
+ throw FutureUtil.wrapToCompletionException(ex);
});
}
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 f1151c5..24cf1d6 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
@@ -2435,6 +2435,9 @@ public class PersistentTopic extends AbstractTopic implements Topic, AddEntryCal
return CompletableFuture.allOf(replicationFuture, dedupFuture, persistentPoliciesFuture,
preCreateSubscriptionForCompactionIfNeeded());
});
+ }).exceptionally(ex -> {
+ log.error("[{}] update namespace polices : {} error", this.getName(), data, ex);
+ throw FutureUtil.wrapToCompletionException(ex);
});
}