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