You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Sravya Tirukkovalur <sr...@cloudera.com> on 2015/11/05 23:13:14 UTC

Re: Review Request 37661: Add describe, show and explain select tests for column level privileges

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

Ship it!


LGTM, thanks for the update Anne!

- Sravya Tirukkovalur


On Oct. 22, 2015, 11:14 p.m., Anne Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37661/
> -----------------------------------------------------------
> 
> (Updated Oct. 22, 2015, 11:14 p.m.)
> 
> 
> Review request for sentry, Lenni Kuff and Sravya Tirukkovalur.
> 
> 
> 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
> -----
> 
>   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 cee05a0 
> 
> 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
> 
>