----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67131/ -----------------------------------------------------------
Review request for sentry, Arjun Mishra, Na Li, and Sergio Pena. Bugs: SENTRY-2174 https://issues.apache.org/jira/browse/SENTRY-2174 Repository: sentry Description ------- SentryAuthorizationProvider should now additionally provide the ACL entries with the permissions that users have along with the permissions for the groups. With the changes proposed in SENTRY-2173, PrivilegeInfo will not only have role to permission mapping. it will also have user to privilege mapping information. SentryAuthorizationProvider should be using the new information added in PrivilegeInfo to add ACL for users. Diffs ----- sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryPermissions.java a88d8e25ad745effe354aa6267252998b189a252 sentry-hdfs/sentry-hdfs-namenode-plugin/src/test/java/org/apache/sentry/hdfs/TestSentryPermissions.java dbce40538cf4721b601cf63b1da713f3d57fc981 Diff: https://reviews.apache.org/r/67131/diff/1/ Testing ------- Added new unit tests and also made sure that all the existing tests pass. Thanks, kalyan kumar kalvagadda