[hive] branch master updated: HIVE-24949: Fail to rename a partition with customized catalog (#2910) (Zhihua Deng reviewed by Zoltan Haindrich)

2022-03-01 Thread dengzh
This is an automated email from the ASF dual-hosted git repository.

dengzh 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 f15e67b  HIVE-24949: Fail to rename a partition with customized 
catalog (#2910) (Zhihua Deng reviewed by Zoltan Haindrich)
f15e67b is described below

commit f15e67b8ff119b41e3257ce6ef5864cece0eaf9b
Author: dengzh 
AuthorDate: Wed Mar 2 09:43:29 2022 +0800

HIVE-24949: Fail to rename a partition with customized catalog (#2910) 
(Zhihua Deng reviewed by Zoltan Haindrich)
---
 .../hcatalog/listener/DbNotificationListener.java  | 44 +++---
 .../ql/metadata/SessionHiveMetaStoreClient.java|  3 +-
 .../hadoop/hive/metastore/DirectSqlUpdateStat.java |  8 +++-
 .../apache/hadoop/hive/metastore/HMSHandler.java   | 10 ++---
 .../hadoop/hive/metastore/HiveAlterHandler.java|  7 ++--
 .../apache/hadoop/hive/metastore/ObjectStore.java  |  3 +-
 .../hadoop/hive/metastore/StatObjectConverter.java | 13 ---
 .../hadoop/hive/metastore/cache/CachedStore.java   |  7 ++--
 .../hive/metastore/VerifyingObjectStore.java   |  1 -
 9 files changed, 49 insertions(+), 47 deletions(-)

diff --git 
a/hcatalog/server-extensions/src/main/java/org/apache/hive/hcatalog/listener/DbNotificationListener.java
 
b/hcatalog/server-extensions/src/main/java/org/apache/hive/hcatalog/listener/DbNotificationListener.java
index 7980d53..fba6827 100644
--- 
a/hcatalog/server-extensions/src/main/java/org/apache/hive/hcatalog/listener/DbNotificationListener.java
+++ 
b/hcatalog/server-extensions/src/main/java/org/apache/hive/hcatalog/listener/DbNotificationListener.java
@@ -141,8 +141,8 @@ import org.apache.hive.hcatalog.data.Pair;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import com.google.common.collect.Lists;
-import static org.apache.hadoop.hive.metastore.Warehouse.DEFAULT_CATALOG_NAME;
 import static 
org.apache.hadoop.hive.metastore.conf.MetastoreConf.ConfVars.EVENT_DB_LISTENER_CLEAN_STARTUP_WAIT_INTERVAL;
+import static 
org.apache.hadoop.hive.metastore.utils.MetaStoreUtils.getDefaultCatalog;
 
 /**
  * An implementation of {@link 
org.apache.hadoop.hive.metastore.MetaStoreEventListener} that
@@ -278,7 +278,7 @@ public class DbNotificationListener extends 
TransactionalMetaStoreEventListener
 NotificationEvent event =
 new NotificationEvent(0, now(), EventType.CREATE_TABLE.toString(),
 msgEncoder.getSerializer().serialize(msg));
-event.setCatName(t.isSetCatName() ? t.getCatName() : DEFAULT_CATALOG_NAME);
+event.setCatName(t.isSetCatName() ? t.getCatName() : 
getDefaultCatalog(conf));
 event.setDbName(t.getDbName());
 event.setTableName(t.getTableName());
 process(event, tableEvent);
@@ -295,7 +295,7 @@ public class DbNotificationListener extends 
TransactionalMetaStoreEventListener
 NotificationEvent event =
 new NotificationEvent(0, now(), EventType.DROP_TABLE.toString(),
 msgEncoder.getSerializer().serialize(msg));
-event.setCatName(t.isSetCatName() ? t.getCatName() : DEFAULT_CATALOG_NAME);
+event.setCatName(t.isSetCatName() ? t.getCatName() : 
getDefaultCatalog(conf));
 event.setDbName(t.getDbName());
 event.setTableName(t.getTableName());
 process(event, tableEvent);
@@ -316,7 +316,7 @@ public class DbNotificationListener extends 
TransactionalMetaStoreEventListener
 new NotificationEvent(0, now(), EventType.ALTER_TABLE.toString(),
 msgEncoder.getSerializer().serialize(msg)
 );
-event.setCatName(after.isSetCatName() ? after.getCatName() : 
DEFAULT_CATALOG_NAME);
+event.setCatName(after.isSetCatName() ? after.getCatName() : 
getDefaultCatalog(conf));
 event.setDbName(after.getDbName());
 event.setTableName(after.getTableName());
 process(event, tableEvent);
@@ -434,7 +434,7 @@ public class DbNotificationListener extends 
TransactionalMetaStoreEventListener
 
 NotificationEvent event = new NotificationEvent(0, now(),
 EventType.ADD_PARTITION.toString(), serializer.serialize(msg));
-event.setCatName(t.isSetCatName() ? t.getCatName() : DEFAULT_CATALOG_NAME);
+event.setCatName(t.isSetCatName() ? t.getCatName() : 
getDefaultCatalog(conf));
 event.setDbName(t.getDbName());
 event.setTableName(t.getTableName());
 process(event, partitionEvent);
@@ -452,7 +452,7 @@ public class DbNotificationListener extends 
TransactionalMetaStoreEventListener
 .buildDropPartitionMessage(t, 
partitionEvent.getPartitionIterator());
 NotificationEvent event = new NotificationEvent(0, now(), 
EventType.DROP_PARTITION.toString(),
 msgEncoder.getSerializer().serialize(msg));
-event.setCatName(t.isSetCatName() ? t.getCatName() : DEFAULT_CATALOG_NAME);
+event.setCatName(t.isSetCatName() ? t.getCatName() : 
getDefaultCatalog(conf));
 event.setDbName(t.getDbName());
 

[hive] branch master updated: HIVE-25665: Checkstyle LGPL files must not be in the release sources/binaries (Peter Vary reviewed by Zoltan Haindrich) (#3063)

2022-03-01 Thread pvary
This is an automated email from the ASF dual-hosted git repository.

pvary 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 4aa4c89  HIVE-25665: Checkstyle LGPL files must not be in the release 
sources/binaries (Peter Vary reviewed by Zoltan Haindrich) (#3063)
4aa4c89 is described below

commit 4aa4c892f35185b1e2159f72d43d9a55abb34c18
Author: pvary 
AuthorDate: Tue Mar 1 23:16:17 2022 +0100

HIVE-25665: Checkstyle LGPL files must not be in the release 
sources/binaries (Peter Vary reviewed by Zoltan Haindrich) (#3063)
---
 checkstyle/checkstyle-noframes-sorted.xsl  | 195 -
 .../checkstyle/checkstyle-noframes-sorted.xsl  | 195 -
 .../checkstyle/checkstyle-noframes-sorted.xsl  | 195 -
 3 files changed, 585 deletions(-)

diff --git a/checkstyle/checkstyle-noframes-sorted.xsl 
b/checkstyle/checkstyle-noframes-sorted.xsl
deleted file mode 100644
index 9c0ac30..000
--- a/checkstyle/checkstyle-noframes-sorted.xsl
+++ /dev/null
@@ -1,195 +0,0 @@
-http://www.w3.org/1999/XSL/Transform; 
version="1.0">
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-   
-   
-   
-.bannercell {
-  border: 0px;
-  padding: 0px;
-}
-body {
-  margin-left: 10;
-  margin-right: 10;
-  font:normal 80% arial,helvetica,sanserif;
-  background-color:#FF;
-  color:#00;
-}
-.a td {
-  background: #efefef;
-}
-.b td {
-  background: #fff;
-}
-th, td {
-  text-align: left;
-  vertical-align: top;
-}
-th {
-  font-weight:bold;
-  background: #ccc;
-  color: black;
-}
-table, th, td {
-  font-size:100%;
-  border: none
-}
-table.log tr td, tr th {
-
-}
-h2 {
-  font-weight:bold;
-  font-size:140%;
-  margin-bottom: 5;
-}
-h3 {
-  font-size:100%;
-  font-weight:bold;
-  background: #525D76;
-  color: white;
-  text-decoration: none;
-  padding: 5px;
-  margin-right: 2px;
-  margin-left: 2px;
-  margin-bottom: 0;
-}
-   
-   
-   
-   
-  
-  
-  
-
-  
-
-   CheckStyle Audit
-   
-   
-   Designed for use with CheckStyle and Ant.
-   
-  
-   
-
-   
-   
-   
-
-   
-   
-   
-
-   
-
-
-   
-
-
-   
-   
-
-
-
-
-   
-   Files
-   
-  
-Name
-Errors
-  
-  
-
-   
-   
-  
-   
-   
-   
-   
-   
-   
-
-
-   
-
-File 
-
-
-   
- Error Description
- Line
-  
-
-  
-   
-
- 
- 
-   
-   
-
-Back to top
-   
-
-
-   
-   Summary
-
-   
-   
-   
-   Files
-   Errors
-   
-   
- 
-   
-   
-   
-   
-   
-
-  
-
-  a
-  b
-
-  
-
-
-
diff --git a/standalone-metastore/checkstyle/checkstyle-noframes-sorted.xsl 
b/standalone-metastore/checkstyle/checkstyle-noframes-sorted.xsl
deleted file mode 100644
index 9c0ac30..000
--- a/standalone-metastore/checkstyle/checkstyle-noframes-sorted.xsl
+++ /dev/null
@@ -1,195 +0,0 @@
-http://www.w3.org/1999/XSL/Transform; 
version="1.0">
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-   
-   
-   
-.bannercell {
-  border: 0px;
-  padding: 0px;
-}
-body {
-  margin-left: 10;
-  margin-right: 10;
-  font:normal 80% arial,helvetica,sanserif;
-  background-color:#FF;
-  color:#00;
-}
-.a td {
-  background: #efefef;
-}
-.b td {
-  background: #fff;
-}
-th, td {
-  text-align: left;
-  vertical-align: top;
-}
-th {
-  font-weight:bold;
-  background: #ccc;
-  color: black;
-}
-table, th, td {
-  font-size:100%;
-  border: none
-}
-table.log tr td, tr th {
-
-}
-h2 {
-  font-weight:bold;
-  font-size:140%;
-  margin-bottom: 5;
-}
-h3 {
-  font-size:100%;
-  font-weight:bold;
-  background: #525D76;
-  color: white;
-  text-decoration: none;
-  padding: 5px;
-  

[hive] branch master updated (bf69b32 -> b63dab1)

2022-03-01 Thread kgyrtkirk
This is an automated email from the ASF dual-hosted git repository.

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


from bf69b32  HIVE-25981: Avoid checking for archived parts in analyze 
table. (#3052). (Ayush Saxena, reviewed by  Rajesh Balamohan and Peter Vary)
 add b63dab1  HIVE-23556: Support hive.metastore.limit.partition.request 
for get_partitions_ps (#3021) (Benny Zhu reviewed by Zoltan Haindrich)

No new revisions were added by this update.

Summary of changes:
 .../hcatalog/listener/DummyRawStoreFailEvent.java  |  6 
 .../hadoop/hive/metastore/conf/MetastoreConf.java  |  4 ++-
 .../apache/hadoop/hive/metastore/HMSHandler.java   | 31 
 .../apache/hadoop/hive/metastore/ObjectStore.java  | 33 ++
 .../org/apache/hadoop/hive/metastore/RawStore.java | 14 +
 .../hadoop/hive/metastore/cache/CachedStore.java   |  6 
 .../metastore/DummyRawStoreControlledCommit.java   |  6 
 .../metastore/DummyRawStoreForJdoConnection.java   |  6 
 .../hadoop/hive/metastore/TestHiveMetaStore.java   | 26 +++--
 .../hadoop/hive/metastore/TestObjectStore.java | 12 
 .../hive/metastore/client/TestListPartitions.java  | 11 
 11 files changed, 145 insertions(+), 10 deletions(-)