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

Reply via email to