Change the string concatenation method to use a placeholder approach …

…for logging. (#298)
ExpediaGroup · Oct 12, 2023 · dc2b8b3 · dc2b8b3
1 parent 72e0790
commit dc2b8b3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...e-core/src/main/java/com/hotels/bdp/waggledance/client/DefaultMetaStoreClientFactory.java b/...e-core/src/main/java/com/hotels/bdp/waggledance/client/DefaultMetaStoreClientFactory.java
@@ -196,7 +196,7 @@ private void genToken() throws Throwable {
       if (delegationToken == null && (currUser = UserGroupInformation.getCurrentUser())
               != UserGroupInformation.getLoginUser()) {
 
-        log.info(String.format("set %s delegation token", currUser.getShortUserName()));
+        log.info("set {} delegation token", currUser.getShortUserName());
         String token = TokenWrappingHMSHandler.getToken();
         setTokenStr2Ugi(currUser, token);
         delegationToken = token;

diff --git a/...e-dance-core/src/main/java/com/hotels/bdp/waggledance/server/TokenWrappingHMSHandler.java b/...e-dance-core/src/main/java/com/hotels/bdp/waggledance/server/TokenWrappingHMSHandler.java
@@ -82,7 +82,7 @@ public Object invoke(Object proxy, Method method, Object[] args) throws Throwabl
 
               String shortName = currUser.getShortUserName();
               token = baseHandler.get_delegation_token(shortName, shortName);
-              log.info(String.format("get delegation token by user %s", shortName));
+              log.info("get delegation token by user {}", shortName);
               tokens.set(token);
             }
             return method.invoke(baseHandler, args);