You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2014/06/13 18:09:23 UTC
svn commit: r1602463 -
/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
Author: hashutosh
Date: Fri Jun 13 16:09:23 2014
New Revision: 1602463
URL: http://svn.apache.org/r1602463
Log:
HIVE-7183 : Size of partColumnGrants should be checked in ObjectStore#removeRole() (Suyeon Lee via Ashutosh Chauhan)
Modified:
hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
Modified: hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
URL: http://svn.apache.org/viewvc/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java?rev=1602463&r1=1602462&r2=1602463&view=diff
==============================================================================
--- hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java (original)
+++ hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java Fri Jun 13 16:09:23 2014
@@ -3167,7 +3167,7 @@ public class ObjectStore implements RawS
}
List<MPartitionColumnPrivilege> partColumnGrants = listPrincipalAllPartitionColumnGrants(
mRol.getRoleName(), PrincipalType.ROLE);
- if (tblColumnGrants.size() > 0) {
+ if (partColumnGrants.size() > 0) {
pm.deletePersistentAll(partColumnGrants);
}
// finally remove the role