You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by ma...@apache.org on 2022/06/11 00:21:50 UTC

[pulsar] branch master updated: [fix][admin] Fix get-publish-rete Admin API handle exception behavior (#16001)

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

mattisonchao 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 addfbd351c9 [fix][admin] Fix get-publish-rete Admin API handle exception behavior (#16001)
addfbd351c9 is described below

commit addfbd351c9b43995f252bd42123b236451ef258
Author: Kai Wang <kw...@streamnative.io>
AuthorDate: Sat Jun 11 08:21:40 2022 +0800

    [fix][admin] Fix get-publish-rete Admin API handle exception behavior (#16001)
---
 .../src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java    | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java
index 00106dd98c9..f55ebb66837 100644
--- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java
+++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java
@@ -773,9 +773,10 @@ public class Namespaces extends NamespacesBase {
             @PathParam("namespace") String namespace) {
         validateNamespaceName(property, namespace);
         internalGetPublishRateAsync()
-                .thenAccept(publishRate -> asyncResponse.resume(publishRate))
+                .thenAccept(asyncResponse::resume)
                 .exceptionally(ex -> {
                     log.error("Failed to get publish rate for namespace {}", namespaceName, ex);
+                    resumeAsyncResponseExceptionally(asyncResponse, ex);
                     return null;
                 });
     }