----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67134/ -----------------------------------------------------------
(Updated May 18, 2018, 11:54 p.m.) Review request for sentry, kalyan kumar kalvagadda, Na Li, Steve Moist, and Sergio Pena. Changes ------- Updated diff post feedback Repository: sentry Description ------- Currently Sentry doesn't support Hive command to show privileges on authorizables without mentioning any role or user name Diffs (updated) ----- sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/SentryHiveAuthorizationTaskFactoryImpl.java 23246c903 sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java fc2427cbf sentry-binding/sentry-binding-hive/src/test/java/org/apache/sentry/binding/hive/TestSentryHiveAuthorizationTaskFactory.java 2e3fd7f36 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java cafe2b597 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestShowGrants.java PRE-CREATION Diff: https://reviews.apache.org/r/67134/diff/3/ Changes: https://reviews.apache.org/r/67134/diff/2-3/ Testing ------- $ mvn -f sentry-binding/pom.xml test $ mvn -f sentry-provider/pom.xml test Thanks, Arjun Mishra
