----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66898/ -----------------------------------------------------------
(Updated May 4, 2018, 8:23 p.m.) Review request for sentry, kalyan kumar kalvagadda, Na Li, Steve Moist, and Sergio Pena. Changes ------- Post feedback from Ben and Sergio 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 Diff: https://reviews.apache.org/r/66898/diff/2/ Changes: https://reviews.apache.org/r/66898/diff/1-2/ Testing ------- Thanks, Arjun Mishra