-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67087/
-----------------------------------------------------------
Review request for sentry, Arjun Mishra, Na Li, and Sergio Pena.
Bugs: SENTRY-2171
https://issues.apache.org/jira/browse/SENTRY-2171
Repository: sentry
Description
-------
When permissions full snapshot is taken by sentry it should include all the
owner privileges added for user and roles.
Implicit privileges added for the users should be added as user privileges in
the full snapshot.
Here are the changes needed.
PermissionsImage should now hold permissions to users
Translate the "OWNER" permission granted to role/user to appropriate permission
Create PermissionsUpdate with the translated permissions.
Diffs
-----
sentry-core/sentry-core-model-db/src/main/java/org/apache/sentry/core/model/db/AccessConstants.java
a8e8bb1543041d356767818d25d7c7cb32cd909b
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
458117020b5204187268a4d9742d00e7fe3925d3
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
679a097678dcaaf51d3ddf69360729212546432e
Diff: https://reviews.apache.org/r/67087/diff/1/
Testing
-------
Made sure all the exiistig tests passed. Also added new test to verify the
change made.
Thanks,
kalyan kumar kalvagadda
