(hive) branch master updated: HIVE-27979: HMS alter_partitions log adds table name (#4978)(dzcxzl,reviewed by Ayush Saxena, 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 529db3968fd HIVE-27979: HMS alter_partitions log adds table name 
(#4978)(dzcxzl,reviewed by Ayush Saxena, Butao Zhang)
529db3968fd is described below

commit 529db3968fd30cabd308908a1d24b04ffb4498e1
Author: cxzl25 <3898450+cxz...@users.noreply.github.com>
AuthorDate: Sat Jan 20 14:06:07 2024 +0800

    HIVE-27979: HMS alter_partitions log adds table name 
(#4978)(dzcxzl,reviewed by Ayush Saxena, Butao Zhang)
---
 .../src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java     | 3 ++-
 .../org/apache/hadoop/hive/metastore/utils/MetaStoreServerUtils.java   | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java
 
b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java
index 54eb8b3c256..a4367d5716a 100644
--- 
a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java
+++ 
b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java
@@ -5919,7 +5919,8 @@ public class HMSHandler extends FacebookBase implements 
IHMSHandler {
 
     if (LOG.isInfoEnabled()) {
       for (Partition tmpPart : new_parts) {
-        LOG.info("New partition values:" + tmpPart.getValues());
+        LOG.info("New partition values: catalog: {} database: {} table: {} 
partition: {}",
+                catName, db_name, tbl_name, tmpPart.getValues());
       }
     }
     // all partitions are altered atomically
diff --git 
a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/MetaStoreServerUtils.java
 
b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/MetaStoreServerUtils.java
index f4afe2aa65d..3c9e0960bc1 100644
--- 
a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/MetaStoreServerUtils.java
+++ 
b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/MetaStoreServerUtils.java
@@ -663,7 +663,8 @@ public class MetaStoreServerUtils {
     if (!madeDir) {
       // The partition location already existed and may contain data. Lets try 
to
       // populate those statistics that don't require a full scan of the data.
-      LOG.info("Updating partition stats fast for: {}", part.getTableName());
+      LOG.info("Updating partition stats fast for: catalog: {} database: {} 
table: {} partition: {}",
+              part.getCatName(), part.getDbName(), part.getTableName(), 
part.getCurrent().getValues());
       List<FileStatus> fileStatus = 
wh.getFileStatusesForLocation(part.getLocation());
       // TODO: this is invalid for ACID tables, and we cannot access AcidUtils 
here.
       populateQuickStats(fileStatus, params);