----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66898/ -----------------------------------------------------------
(Updated May 4, 2018, 9:26 p.m.) Review request for sentry, kalyan kumar kalvagadda, Na Li, Steve Moist, and Sergio Pena. Changes ------- Modified sentrystore grant method to match with revoke method Repository: sentry Description ------- We can grant privileges to URI's with EMPTY or NULL locations. This should not be allowed Diffs (updated) ----- sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java 3ac49fa6a sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryPlugin.java 8485ca37f sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java 8ac3c0d51 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java b41002783 Diff: https://reviews.apache.org/r/66898/diff/4/ Changes: https://reviews.apache.org/r/66898/diff/3-4/ Testing ------- $ mvn -f sentry-binding/sentry-binding-hive/pom.xml test $ mvn -f sentry-hdfs/sentry-hdfs-service/pom.xml test $ mvn -f sentry-provider/sentry-provider-db/pom.xml test Thanks, Arjun Mishra
