(hive) branch master updated: HIVE-28857. Fix logging for invalid partition path (#5725)(Cheng Pan, reviewed by Zsolt Miskolczi, Butao Zhang)

[zhangbutao]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new d703ea1a3f4 HIVE-28857. Fix logging for invalid partition path 
(#5725)(Cheng Pan, reviewed by Zsolt Miskolczi, Butao Zhang)
d703ea1a3f4 is described below

commit d703ea1a3f4977b8bdde8b674e2c40b1be3f2ba9
Author: Cheng Pan <cheng...@apache.org>
AuthorDate: Mon Apr 7 17:30:04 2025 +0800

    HIVE-28857. Fix logging for invalid partition path (#5725)(Cheng Pan, 
reviewed by Zsolt Miskolczi, Butao Zhang)
---
 .../src/main/java/org/apache/hadoop/hive/metastore/Warehouse.java    | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/Warehouse.java
 
b/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/Warehouse.java
index 57320d44892..9fd9370a2a4 100755
--- 
a/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/Warehouse.java
+++ 
b/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/Warehouse.java
@@ -680,9 +680,10 @@ public static LinkedHashMap<String, String> 
makeSpecFromName(String name)
     return partSpec;
   }
 
-  public static boolean makeSpecFromName(Map<String, String> partSpec, Path 
currPath,
+  public static boolean makeSpecFromName(Map<String, String> partSpec, Path 
path,
       Set<String> requiredKeys) {
     List<String[]> kvs = new ArrayList<>();
+    Path currPath = path;
     do {
       String component = currPath.getName();
       Matcher m = pat.matcher(component);
@@ -706,7 +707,7 @@ public static boolean makeSpecFromName(Map<String, String> 
partSpec, Path currPa
       partSpec.put(key, kvs.get(i - 1)[1]);
     }
     if (requiredKeys == null || requiredKeys.isEmpty()) return true;
-    LOG.warn("Cannot create partition spec from " + currPath + "; missing keys 
" + requiredKeys);
+    LOG.warn("Cannot create partition spec from {}; missing keys {}", path, 
requiredKeys);
     return false;
   }