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

ngangam pushed a commit to branch branch-3.1
in repository https://gitbox.apache.org/repos/asf/hive.git

commit b91aec6729f4ed3f9d1f56ae7ae122c72249b22b
Author: Naveen Gangam <[email protected]>
AuthorDate: Fri Feb 4 11:19:13 2022 -0500

    TestPartition test failure in HMS tests (Naveen Gangam)
---
 .../java/org/apache/hadoop/hive/metastore/ObjectStore.java  | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git 
a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
 
b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
index f32e497..99fc74e 100644
--- 
a/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
+++ 
b/standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
@@ -3227,12 +3227,13 @@ public class ObjectStore implements RawStore, 
Configurable {
           max_parts, null, queryWrapper);
       for (Object o : parts) {
         Partition part = convertToPart((MPartition) o);
-        // set auth privileges
-        String partName = 
Warehouse.makePartName(this.convertToFieldSchemas(mtbl
-            .getPartitionKeys()), part.getValues());
-        PrincipalPrivilegeSet partAuth = getPartitionPrivilegeSet(catName, 
db_name,
-            tbl_name, partName, userName, groupNames);
-        part.setPrivileges(partAuth);
+        if (getauth) {
+          // set auth privileges
+          String partName = 
Warehouse.makePartName(this.convertToFieldSchemas(mtbl.getPartitionKeys()), 
part.getValues());
+          PrincipalPrivilegeSet partAuth =
+              getPartitionPrivilegeSet(catName, db_name, tbl_name, partName, 
userName, groupNames);
+          part.setPrivileges(partAuth);
+        }
         partitions.add(part);
       }
       success = commitTransaction();

Reply via email to