(helix) branch master updated: Optimize logging in LogUtil without string format (#2880)

[jxue]

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

jxue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/helix.git


The following commit(s) were added to refs/heads/master by this push:
     new 7ed0ef92a Optimize logging in LogUtil without string format (#2880)
7ed0ef92a is described below

commit 7ed0ef92a719103f90fdd6651294b1d56bea230e
Author: Junkai Xue <junkai....@gmail.com>
AuthorDate: Fri Aug 9 18:02:56 2024 -0700

    Optimize logging in LogUtil without string format (#2880)
    
    Optimize logging in LogUtil without string format
---
 .../main/java/org/apache/helix/controller/LogUtil.java   | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/helix-core/src/main/java/org/apache/helix/controller/LogUtil.java 
b/helix-core/src/main/java/org/apache/helix/controller/LogUtil.java
index ec96568aa..48b8aa765 100644
--- a/helix-core/src/main/java/org/apache/helix/controller/LogUtil.java
+++ b/helix-core/src/main/java/org/apache/helix/controller/LogUtil.java
@@ -23,34 +23,34 @@ import org.slf4j.Logger;
 
 public class LogUtil {
   public static void logInfo(Logger logger, String eventInfo, String message) {
-    logger.info(String.format("Event %s : %s", eventInfo, message));
+    logger.info("Event {} : {}", eventInfo, message);
   }
 
   public static void logWarn(Logger logger, String eventInfo, String message) {
-    logger.warn(String.format("Event %s : %s", eventInfo, message));
+    logger.warn("Event {} : {}", eventInfo, message);
   }
 
   public static void logError(Logger logger, String eventInfo, String message) 
{
-    logger.error(String.format("Event %s : %s", eventInfo, message));
+    logger.error("Event {} : {}", eventInfo, message);
   }
 
   public static void logDebug(Logger logger, String eventInfo, String message) 
{
-    logger.debug(String.format("Event %s : %s", eventInfo, message));
+    logger.debug("Event {} : {}", eventInfo, message);
   }
 
   public static void logInfo(Logger logger, String eventInfo, String message, 
Exception e) {
-    logger.info(String.format("Event %s : %s", eventInfo, message), e);
+    logger.info("Event {} : {}", eventInfo, message, e);
   }
 
   public static void logWarn(Logger logger, String eventInfo, String message, 
Exception e) {
-    logger.warn(String.format("Event %s : %s", eventInfo, message), e);
+    logger.warn("Event {} : {}", eventInfo, message, e);
   }
 
   public static void logError(Logger logger, String eventInfo, String message, 
Exception e) {
-    logger.error(String.format("Event %s : %s", eventInfo, message), e);
+    logger.error("Event {} : {}", eventInfo, message, e);
   }
 
   public static void logDebug(Logger logger, String eventInfo, String message, 
Exception e) {
-    logger.debug(String.format("Event %s : %s", eventInfo, message), e);
+    logger.debug("Event {} : {}", eventInfo, message, e);
   }
 }