[GitHub] [pulsar] codelipenghui commented on a change in pull request #7725: Change some WebApplicationException log level to debug

[GitBox <gi...@apache.org>]

codelipenghui commented on a change in pull request #7725:
URL: https://github.com/apache/pulsar/pull/7725#discussion_r464314269



##########
File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java
##########
@@ -599,6 +599,11 @@ protected PartitionedTopicMetadata internalGetPartitionedMetadata(boolean author
     protected void internalDeletePartitionedTopic(AsyncResponse asyncResponse, boolean authoritative, boolean force) {
         try {
             validateWriteOperationOnTopic(authoritative);
+        } catch (WebApplicationException wae) {
+            log.debug("[{}] Failed to delete partitioned topic {}, redirecting to other brokers.",

Review comment:
       Please add check for the debug is enabled
   if(log.isDebugEnable) {
     log.debug("....")
   }




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org