-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68926/
-----------------------------------------------------------
(Updated Oct. 20, 2018, 1:58 a.m.)
Review request for sentry and kalyan kumar kalvagadda.
Changes
-------
Addressed comments and add unit tests.
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-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/SentryUtils.java
b225af0bc4e8c472e2ee17677a74525b882e214c
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryPlugin.java
b8f5ce7dbc8cd2dd6317f02cc09a225ef460acf4
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/model/MSentryPrivilege.java
eba40d2ebd9778ddf74df0d35ad8b22500717f73
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
b387a22e40b8958395e1c12af12272b89a778726
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreInterface.java
6217719b8cf0c120b47a1612f6b3f0bbba98f724
sentry-service/sentry-service-server/src/test/java/org/apache/sentry/api/service/thrift/SentryPolicyStoreProcessorTestUtils.java
PRE-CREATION
sentry-service/sentry-service-server/src/test/java/org/apache/sentry/api/service/thrift/TestSentryPolicyStoreProcessor.java
16ff79e2e4258c20a46ffcf162872b5b20d97aba
Diff: https://reviews.apache.org/r/68926/diff/3/
Changes: https://reviews.apache.org/r/68926/diff/2-3/
Testing
-------
Thanks,
Sergio Pena
