-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67803/#review205745
-----------------------------------------------------------




sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestOwnerPrivileges.java
Lines 40 (patched)
<https://reviews.apache.org/r/67803/#comment288616>

    1. Please add tests for negative cases. Where other users who have create 
permissions can only create new tables and use them but not able to access 
tables created by other users.
    
    2. Also add cases where owner privileges are not granted to users belonging 
to sentry admim group on creation of tables and databases.



sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestOwnerPrivileges.java
Lines 45-54 (patched)
<https://reviews.apache.org/r/67803/#comment288614>

    Some of these varables are not used. Could you please remove it.



sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestOwnerPrivileges.java
Lines 79 (patched)
<https://reviews.apache.org/r/67803/#comment288615>

    Please fix the typo "Verfity". You need to fix it in couple of places.


- kalyan kumar kalvagadda


On July 5, 2018, 4:03 p.m., Na Li wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67803/
> -----------------------------------------------------------
> 
> (Updated July 5, 2018, 4:03 p.m.)
> 
> 
> Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Sergio 
> Pena.
> 
> 
> Bugs: sentry-2247
>     https://issues.apache.org/jira/browse/sentry-2247
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> add test case to verify the owner privileges are added at db and table 
> creation. Some testing cases have to be ignore until sentry-2291 is done
> 
> Fix multiple issues found by the new test cases:
> 1. The server name of owner privilege was not set at server side
> 2. When dropping privilege, if a privilege has no role, it is not dropped
> 3. When dropping privilege, if it has grant option being true, it is not 
> dropped.
> 
> 
> Diffs
> -----
> 
>   
> sentry-service/sentry-service-server/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyStoreProcessor.java
>  98f2e29 
>   
> sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
>  ff0b4c0 
>   
> sentry-service/sentry-service-server/src/test/java/org/apache/sentry/api/service/thrift/TestSentryPolicyStoreProcessor.java
>  14a0fd8 
>   
> sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestOwnerPrivileges.java
>  PRE-CREATION 
>   
> sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestOwnerPrivilegesWithGrantOption.java
>  PRE-CREATION 
>   
> sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java
>  100219b 
> 
> 
> Diff: https://reviews.apache.org/r/67803/diff/4/
> 
> 
> Testing
> -------
> 
> test cases in TestOwnerPrivileges and TestOwnerPrivilegesWithGrantOption 
> passed
> 
> 
> Thanks,
> 
> Na Li
> 
>

Reply via email to