----------------------------------------------------------- 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 > >
