-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22494/
-----------------------------------------------------------
(Updated June 12, 2014, 10:27 p.m.)
Review request for sentry, Jarek Cecho, Prasad Mujumdar, and Sravya
Tirukkovalur.
Changes
-------
Updating patch to clean up a bit of the code and added comments..
Bugs: SENTRY-281
https://issues.apache.org/jira/browse/SENTRY-281
Repository: sentry
Description
-------
Fix for SENTRY-281 :
A revoke on a parent privilege should trickle to the child.
For eg :
1) GRANT SELECT on DATABASE db1 to ROLE role1;
2) REVOKE ALL on SERVER server1 from ROLE role1;
Should result in zero privileges for role1
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
91669d6
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java
5d7428a
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java
44f41be
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbEndToEnd.java
634dd52
Diff: https://reviews.apache.org/r/22494/diff/
Testing
-------
Tested on sentry-hive-tests
Thanks,
Arun Suresh
