[ 
https://issues.apache.org/jira/browse/HIVE-7183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14027773#comment-14027773
 ] 

Swarnim Kulkarni commented on HIVE-7183:
----------------------------------------

[~suyeon1222] I am only a contributor on the project and not a committer. So my 
vote counts towards being a non-binding. A committer's vote is considered as a 
binding vote which you would need to get this patch accepted. For further 
information, refer to [1]

[1] 
https://cwiki.apache.org/confluence/display/Hive/Proposed+Changes+to+Hive+Bylaws+for+Submodule+Committers#ProposedChangestoHiveBylawsforSubmoduleCommitters-DecisionMaking

> Size of partColumnGrants should be checked in ObjectStore#removeRole()
> ----------------------------------------------------------------------
>
>                 Key: HIVE-7183
>                 URL: https://issues.apache.org/jira/browse/HIVE-7183
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Priority: Minor
>         Attachments: HIVE-7183.patch
>
>
> Here is related code:
> {code}
>         List<MPartitionColumnPrivilege> partColumnGrants = 
> listPrincipalAllPartitionColumnGrants(
>             mRol.getRoleName(), PrincipalType.ROLE);
>         if (tblColumnGrants.size() > 0) {
>           pm.deletePersistentAll(partColumnGrants);
> {code}
> Size of tblColumnGrants is currently checked.
> Size of partColumnGrants should be checked instead.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to