[GitHub] [bookkeeper] Shoothzj commented on a change in pull request #2605: Fix logger member not correct、some javadoc problem

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

Shoothzj commented on a change in pull request #2605:
URL: https://github.com/apache/bookkeeper/pull/2605#discussion_r578037844



##########
File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RackawareEnsemblePlacementPolicyImpl.java
##########
@@ -269,9 +269,9 @@ public RackawareEnsemblePlacementPolicyImpl initialize(ClientConfiguration conf,
                 }
             } catch (RuntimeException re) {
                 if (!conf.getEnforceMinNumRacksPerWriteQuorum()) {
-                    LOG.error("Failed to initialize DNS Resolver {}, used default subnet resolver : {}",
+                    LOG.error("Failed to initialize DNS Resolver {}, used default subnet resolver : {} {}",

Review comment:
       I remeber only one exception args can print without matching `{}`, I remove the re.getMessage.




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