-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19560/
-----------------------------------------------------------
(Updated March 24, 2014, 5:24 p.m.)
Review request for sentry, Brock Noland and Shreepadma Venugopalan.
Changes
-------
Changes per review feedback.
Bugs: SENTRY-137
https://issues.apache.org/jira/browse/SENTRY-137
Repository: sentry
Description
-------
Put privilege scopes in SentryConstants [SERVER, DB, TABLE, URI]
Ensure the SentryPolicyStoreClient client is setting this properties.
Validate this property when on the grant/revoke privilege commands in
SentryStore
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClient.java
464569c
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java
9562783
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
1b36690
Diff: https://reviews.apache.org/r/19560/diff/
Testing
-------
sentry-provider-db tests pass with the patch.
Thanks,
Prasad Mujumdar
