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

(Updated Sept. 22, 2015, 11:57 p.m.)


Review request for sentry, Lenni Kuff and Sravya Tirukkovalur.


Changes
-------

1. addressed Sravya's comments;
2. added negative test cases;
3. added test partitions;
4. some metadata operations require table level privileges.


Bugs: SENTRY-742
    https://issues.apache.org/jira/browse/SENTRY-742


Repository: sentry


Description
-------

Add a couple of test cases to cover column level privileges when user do 
DESCRIBE, SHOW AND EXPLAIN SELECT cases.


Diffs (updated)
-----

  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbColumnLevelMetaDataOps.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/PrivilegeResultSet.java
 cee05a010762dda96e08d48a1b0c03dadffb2478 

Diff: https://reviews.apache.org/r/37661/diff/


Testing
-------

-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running org.apache.sentry.tests.e2e.dbprovider.TestDbColumnLevelMetaDataOps
Tests run: 6, Failures: 0, Errors: 0, Skipped: 6, Time elapsed: 17.789 sec - in 
org.apache.sentry.tests.e2e.dbprovider.TestDbColumnLevelMetaDataOps

Results :

Tests run: 6, Failures: 0, Errors: 0, Skipped: 6


Thanks,

Anne Yu

Reply via email to