You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by pe...@apache.org on 2022/02/25 08:07:21 UTC

[pulsar] 03/03: fix error log miss stack trace when create tenant fail (#14366)

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

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

commit 214d292acfe502304aaa2e48e3cab6fe73025731
Author: WangJialing <65...@users.noreply.github.com>
AuthorDate: Mon Feb 21 22:22:59 2022 +0800

    fix error log miss stack trace when create tenant fail (#14366)
    
    ### Motivation
    
    When create tenant fail, the error log did not print out the stack trace.
    
    ### Verifying this change
    
    This change is a trivial rework / code cleanup without any test coverage.
    
    ### Documentation
    - [x] `no-need-doc`
    only a fix for error log
    
    (cherry picked from commit b829a4ce121268f55748bbdd6f19ac36129e7dab)
---
 .../main/java/org/apache/pulsar/broker/admin/impl/TenantsBase.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/TenantsBase.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/TenantsBase.java
index 9b04830..4f1bad2 100644
--- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/TenantsBase.java
+++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/TenantsBase.java
@@ -162,12 +162,12 @@ public class TenantsBase extends PulsarWebResource {
                     log.info("[{}] Created tenant {}", clientAppId(), tenant);
                     asyncResponse.resume(Response.noContent().build());
                 }).exceptionally(ex -> {
-                    log.error("[{}] Failed to create tenant {}", clientAppId, tenant, e);
+                    log.error("[{}] Failed to create tenant {}", clientAppId, tenant, ex);
                     asyncResponse.resume(new RestException(ex));
                     return null;
                 });
             }).exceptionally(ex -> {
-                log.error("[{}] Failed to create tenant {}", clientAppId(), tenant, e);
+                log.error("[{}] Failed to create tenant {}", clientAppId(), tenant, ex);
                 asyncResponse.resume(new RestException(ex));
                 return null;
             });