[pulsar] branch branch-2.9 updated: Add log when update namespace policies with error. (#15056)

[technoboy]

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

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


The following commit(s) were added to refs/heads/branch-2.9 by this push:
     new e04c19eec38 Add log when update namespace policies with error. (#15056)
e04c19eec38 is described below

commit e04c19eec38978c58ff38cbcdcb056f94cbe9bb7
Author: Jiwei Guo <techno...@apache.org>
AuthorDate: Fri Apr 29 10:20:31 2022 +0800

    Add log when update namespace policies with error. (#15056)
---
 .../pulsar/broker/service/nonpersistent/NonPersistentTopic.java      | 3 +++
 .../org/apache/pulsar/broker/service/persistent/PersistentTopic.java | 5 ++++-
 2 files changed, 7 insertions(+), 1 deletion(-)

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 472cf1cc200..b9a85ab97c5 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
@@ -1001,6 +1001,9 @@ public class NonPersistentTopic extends AbstractTopic 
implements Topic {
                         }
                         return 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 bdfb2b2e2b2..333438849e6 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
@@ -2496,8 +2496,11 @@ public class PersistentTopic extends AbstractTopic
                 }
 
                 return CompletableFuture.allOf(replicationFuture, dedupFuture, 
persistentPoliciesFuture,
-                        preCreateSubscriptionForCompactionIfNeeded());
+                                preCreateSubscriptionForCompactionIfNeeded());
             });
+        }).exceptionally(ex -> {
+            log.error("[{}] update namespace polices : {} error", 
this.getName(), data, ex);
+            throw FutureUtil.wrapToCompletionException(ex);
         });
     }