[GitHub] [pulsar] tomscut commented on a change in pull request #11880: [Broker] Remove redundant log guards for pulsar-broker

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

tomscut commented on a change in pull request #11880:
URL: https://github.com/apache/pulsar/pull/11880#discussion_r700670687



##########
File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarClusterMetadataTeardown.java
##########
@@ -146,9 +146,7 @@ public static void main(String[] args) throws Exception {
     private static void deleteLedger(BookKeeper bookKeeper, long ledgerId) {
         try {
             bookKeeper.deleteLedger(ledgerId);
-            if (log.isDebugEnabled()) {
-                log.debug("Delete ledger id: {}", ledgerId);
-            }
+            log.debug("Delete ledger id: {}", ledgerId);

Review comment:
       Thank you for your review and suggestions. I will make some changes.




-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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