-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39824/
-----------------------------------------------------------
Review request for sentry, Anne Yu, Lenni Kuff, and Ryan Pridgeon.
Bugs: SENTRY-936
https://issues.apache.org/jira/browse/SENTRY-936
Repository: sentry
Description
-------
Paths which do not correspond to hive objects should not be affected by Sentry
HDFS sync.
We should update getGroup/getUser/getPermission and getACL
Old behavior:
If in prefix + unmanaged:
- Not stale: defaultProvider
- Stale: hive:hive, 771
If managed : hive:hive, 771
New behavior:
If in prefix + unmanaged: defaultProvider
If managed : hive:hive, 771
Diffs
-----
sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationProvider.java
d167183bcac8f56dd58bda0e4ff8406dbd3ad60c
sentry-hdfs/sentry-hdfs-namenode-plugin/src/test/java/org/apache/sentry/hdfs/SentryAuthorizationInfoX.java
4cebed2ef67ff6e4135441a737ed8ad494c84af8
sentry-hdfs/sentry-hdfs-namenode-plugin/src/test/java/org/apache/sentry/hdfs/TestSentryAuthorizationProvider.java
40b803e6be2c98cc97ef0baf67e6736d97208b6f
Diff: https://reviews.apache.org/r/39824/diff/
Testing
-------
Existing tests pass. Added a new unit test for stale case validation.
Thanks,
Sravya Tirukkovalur
