----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67134/ -----------------------------------------------------------
(Updated June 19, 2018, 7:46 p.m.) Review request for sentry, kalyan kumar kalvagadda, Na Li, Steve Moist, and Sergio Pena. 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 051893868 sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java 321701d86 sentry-binding/sentry-binding-hive/src/test/java/org/apache/sentry/binding/hive/TestSentryHiveAuthorizationTaskFactory.java 2e3fd7f36 sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryObjectPrivileges.java PRE-CREATION sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClient.java 6f38ed20d sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java 45dce0e7c 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/14/ Changes: https://reviews.apache.org/r/67134/diff/13-14/ Testing ------- $ mvn -f sentry-binding/pom.xml test $ mvn -f sentry-provider/pom.xml test Thanks, Arjun Mishra
