-----------------------------------------------------------
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
