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

Dapeng Sun commented on SENTRY-529:
-----------------------------------

Hi [~lskuff]]
Thank you for your quick reply, the {{Set<TSentryPrivilege>}} couldn't be not 
set at here.
Here is the preconditions check
https://github.com/apache/incubator-sentry/blob/13e83d6e524e696ef973f251577e93f236e39cd9/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java#L251


> [Unit Test]org.apache.sentry.tests.e2e.hdfs.TestHDFSIntegrationtestEnd2End 
> test fails after SENTRY-74
> -----------------------------------------------------------------------------------------------------
>
>                 Key: SENTRY-529
>                 URL: https://issues.apache.org/jira/browse/SENTRY-529
>             Project: Sentry
>          Issue Type: Bug
>            Reporter: Dapeng Sun
>            Assignee: Dapeng Sun
>            Priority: Blocker
>         Attachments: SENTRY-529.002.patch, SENTRY-529.patch
>
>
> {{SentryHDFSPlugin}} is using {{TSentryPrivilege}} in 
> {{TAlterSentryRoleRevokePrivilegeRequest}} and  
> {{TAlterSentryRoleGrantPrivilegeRequest}} , after SENTRY-74, it have changed 
> to {{Set<TSentryPrivilege>}},adjust {{SentryHDFSPlugin}} for the change.



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

Reply via email to