-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22704/
-----------------------------------------------------------
(Updated June 18, 2014, 9:01 p.m.)
Review request for sentry.
Changes
-------
Added restriction for 'set role' to allowed roles only. Added new test case.
Bugs: SENTRY-305
https://issues.apache.org/jira/browse/SENTRY-305
Repository: sentry
Description
-------
Allow 'show current role' without admin privilege. The admin auth check is
relaxed if you want to list the roles for the requesting user. This is
requested when executing show current role.
Diffs (updated)
-----
sentry-binding/sentry-binding-hive/src/main/java/org/apache/hadoop/hive/ql/exec/SentryGrantRevokeTask.java
122d137
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzBinding.java
b478ec2
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
56fc3df
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClient.java
9e2c200
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java
097056b
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
35ba83a
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java
27ef9ce
Diff: https://reviews.apache.org/r/22704/diff/
Testing
-------
Added new unit test and e2e test
Thanks,
Prasad Mujumdar
