This is an automated email from the ASF dual-hosted git repository. voonhous pushed a commit to branch release-1.2.1 in repository https://gitbox.apache.org/repos/asf/hudi.git
commit ab746b31fd3e6873aee99072f1b9795834ac7103 Author: voonhous <[email protected]> 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. (cherry picked from commit 905fcdda42ac6147594b73c222bb960e540cfb22) --- .../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 2a130d8ee797..fc07512c353b 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);
