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

Prasad Mujumdar commented on SENTRY-494:
----------------------------------------

[~colinma] The patch mostly looks fine. A couple of comments/suggestions -
- Please rebase the patch. The pom.xml change doesn't apply cleanly.
- HiveAuthzPrivilegesMap.java: Remove the test 'todo' comment.
- TestLockPrivileges.java - I am not sure why the test is creating the 
transaction related Hive metastore tables ? Also now that we are compiling with 
latest Hive, is that still relevant ?


> UNLOCK TABLE is not allowed
> ---------------------------
>
>                 Key: SENTRY-494
>                 URL: https://issues.apache.org/jira/browse/SENTRY-494
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: 1.3.0
>         Environment: Sentry 1.3.0 CDH5.1.2
>            Reporter: Grisha Trubetskoy
>            Assignee: Colin Ma
>         Attachments: SENTRY-494.001.patch, SENTRY-494.002.patch
>
>
> {code}
> UNLOCK TABLE grisha_test;
> Error: Error while compiling statement: FAILED: SemanticException No valid 
> privileges (state=42000,code=40000)
> {code}
> Unless I'm missing something, there is no way to specify an unlock privilege 
> in Sentry.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to