(hudi) branch master updated: refactor(sync): use SLF4J parameterized logging instead of string concatenation (#19189)

[vhs]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 905fcdda42ac refactor(sync): use SLF4J parameterized logging instead 
of string concatenation (#19189)
905fcdda42ac is described below

commit 905fcdda42ac6147594b73c222bb960e540cfb22
Author: voonhous <voonho...@gmail.com>
AuthorDate: Sun Jul 5 12:04:52 2026 +0800

    refactor(sync): use SLF4J parameterized logging instead of string 
concatenation (#19189)
    
    Convert log string concatenation to {} placeholders in HiveSyncTool and
    QueryBasedDDLExecutor. Mechanical and behavior-preserving.
---
 .../src/main/java/org/apache/hudi/hive/HiveSyncTool.java              | 2 +-
 .../src/main/java/org/apache/hudi/hive/ddl/QueryBasedDDLExecutor.java | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HiveSyncTool.java 
b/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HiveSyncTool.java
index 85e59ea7a240..763a5555920a 100644
--- 
a/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HiveSyncTool.java
+++ 
b/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HiveSyncTool.java
@@ -555,7 +555,7 @@ public class HiveSyncTool extends HoodieSyncTool implements 
AutoCloseable {
 
     List<String> touchPartitions = 
config.getBoolean(META_SYNC_TOUCH_PARTITIONS_ENABLED) ? 
filterPartitions(partitionEventList, PartitionEventType.TOUCH) : 
Collections.emptyList();
     if (!touchPartitions.isEmpty()) {
-      log.info("Touch Partitions " + touchPartitions);
+      log.info("Touch Partitions {}", touchPartitions);
       syncClient.touchPartitionsToTable(tableName, touchPartitions);
     }
 
diff --git 
a/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/ddl/QueryBasedDDLExecutor.java
 
b/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/ddl/QueryBasedDDLExecutor.java
index 7f776f2f7a04..a4e3ee309154 100644
--- 
a/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/ddl/QueryBasedDDLExecutor.java
+++ 
b/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/ddl/QueryBasedDDLExecutor.java
@@ -205,10 +205,10 @@ public abstract class QueryBasedDDLExecutor implements 
DDLExecutor {
   @Override
   public void touchPartitionsToTable(String tableName, List<String> 
touchPartitions) {
     if (touchPartitions.isEmpty()) {
-      log.info("No partitions to touch for " + tableName);
+      log.info("No partitions to touch for {}", tableName);
       return;
     }
-    log.info("Touching partitions " + touchPartitions.size() + " on " + 
tableName);
+    log.info("Touching partitions {} on {}", touchPartitions.size(), 
tableName);
     List<String> sqls = constructPartitionAlterStatements(tableName, 
touchPartitions, PartitionAlterType.TOUCH);
     for (String sql : sqls) {
       runSQL(sql);