-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68926/
-----------------------------------------------------------

(Updated Oct. 5, 2018, 2:23 p.m.)


Review request for sentry and kalyan kumar kalvagadda.


Bugs: SENTRY-2372
    https://issues.apache.org/jira/browse/SENTRY-2372


Repository: sentry


Description
-------

Make the grant option check on the SentryPolicyStoreProcessor so that it gets 
the group information from it. The SentryStore has the new method to check the 
grant option in the DB only.


Diffs (updated)
-----

  
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyStoreProcessor.java
 709434c388689b63d5db7d71cd6cc47952d647bf 
  
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
 1722109e381bc7be81a4673d12c1ac9f81195c47 
  
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreInterface.java
 3a68eb6bfa0224d1bce17f0bfccb008ab9b291f2 
  
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreUtils.java
 PRE-CREATION 
  
sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/service/persistent/TestHMSFollowerSentryStoreIntegration.java
 a8868364437ac637ba4a9ad551408de0a9304984 
  
sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
 a299e008cc0df92f3d692024c7794aa494b64d2d 
  
sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStoreToAuthorizable.java
 25f94fa05b05abf8c1dbc33e97e5e88ae01794e4 


Diff: https://reviews.apache.org/r/68926/diff/2/

Changes: https://reviews.apache.org/r/68926/diff/1-2/


Testing
-------


Thanks,

Sergio Pena

Reply via email to