You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Brock Noland <br...@cloudera.com> on 2014/03/12 14:22:19 UTC

Review Request 19118: SENTRY-132 - Implement APIs that grant/revoke roles to groups

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

Review request for sentry and Shreepadma Venugopalan.


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


Repository: sentry


Description
-------

Adds Alter role drop/add groups


Diffs
-----

  sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryGroup.java 15982a3 
  sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPrivilege.java 9642689 
  sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryRole.java 9559c57 
  sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java 5df6657 
  sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java 78e0a87 
  sentry-provider/sentry-provider-db/src/main/resources/sentry_policy_service.thrift d6e05b7 
  sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java PRE-CREATION 

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


Testing
-------

New unit tests TestSentryStore.


Thanks,

Brock Noland


Re: Review Request 19118: SENTRY-132 - Implement APIs that grant/revoke roles to groups

Posted by Shreepadma Venugopalan <sh...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19118/#review36986
-----------------------------------------------------------

Ship it!


Ship It!

- Shreepadma Venugopalan


On March 12, 2014, 1:22 p.m., Brock Noland wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19118/
> -----------------------------------------------------------
> 
> (Updated March 12, 2014, 1:22 p.m.)
> 
> 
> Review request for sentry and Shreepadma Venugopalan.
> 
> 
> Bugs: SENTRY-132
>     https://issues.apache.org/jira/browse/SENTRY-132
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Adds Alter role drop/add groups
> 
> 
> Diffs
> -----
> 
>   sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryGroup.java 15982a3 
>   sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPrivilege.java 9642689 
>   sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryRole.java 9559c57 
>   sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java 5df6657 
>   sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java 78e0a87 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry_policy_service.thrift d6e05b7 
>   sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/19118/diff/
> 
> 
> Testing
> -------
> 
> New unit tests TestSentryStore.
> 
> 
> Thanks,
> 
> Brock Noland
> 
>