----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69415/#review210899 -----------------------------------------------------------
Ship it! Ship It! - Haley Reeve On Nov. 26, 2018, 7:36 p.m., Arjun Mishra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69415/ > ----------------------------------------------------------- > > (Updated Nov. 26, 2018, 7:36 p.m.) > > > Review request for sentry, Haley Reeve, kalyan kumar kalvagadda, Na Li, and > Sergio Pena. > > > Bugs: SENTRY-2463 > https://issues.apache.org/jira/browse/SENTRY-2463 > > > Repository: sentry > > > Description > ------- > > Right now if we initially grant ALL and revoke SELECT or INSERT, the > privilege gets "modified" to INSERT or SELECT. However, conversely if we > initially grant SELECT or INSERT and revoke ALL, no privileges are dropped > > > Diffs > ----- > > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java > e2d6c85ac > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java > 6a7c1f392 > > > Diff: https://reviews.apache.org/r/69415/diff/1/ > > > Testing > ------- > > mvn -f sentry-tests/sentry-tests-hive/pom.xml test -Dtest=TestDatabaseProvider > mvn -f sentry-service/sentry-service-server/pom.xml test > -Dtest=TestSentryStore > > > Thanks, > > Arjun Mishra > >
