----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68367/ -----------------------------------------------------------
(Updated Aug. 17, 2018, 2:56 a.m.) Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Sergio Pena. Bugs: sentry-2354 https://issues.apache.org/jira/browse/sentry-2354 Repository: sentry Description ------- When multiple privileges are required on a Hive operation, return the privilege that failed access check and the required privileges not checked yet (they may fail the access check, or may not. We don't check those privileges to avoid unnecessary overhead) Diffs (updated) ----- sentry-binding/sentry-binding-hive-common/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzBinding.java 6a1556f sentry-binding/sentry-binding-hive/src/test/java/org/apache/sentry/binding/hive/TestHiveAuthzBindings.java 3bbf6fb sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/AuthorizationProvider.java aecfe5b sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/NoAuthorizationProvider.java 61400ca sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java 1e1aa63 Diff: https://reviews.apache.org/r/68367/diff/2/ Changes: https://reviews.apache.org/r/68367/diff/1-2/ Testing ------- Add test case that verifies the behavior stated above. All test cases in TestHiveAuthzBindings passed Thanks, Na Li