----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67006/ -----------------------------------------------------------
(Updated May 9, 2018, 7:13 p.m.) Review request for sentry, Na Li and Sergio Pena. Changes ------- Added new comment. Bugs: SENTRY-2173 https://issues.apache.org/jira/browse/SENTRY-2173 Repository: sentry Description ------- PrivilegeInfo should be updated to hold the permissions added to the users. PrivilegeInfo currently holds roles <=> privileges mapping. This data structure should be made generic so that it could hold the permissions of either roles/users on a given object. Diffs (updated) ----- sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryPermissions.java 3b3a82e8fce7986bd046e96192105952aa6fa796 sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPermissions.java 2ad74403b279fc2feafda361c92bf6a3d259d0eb Diff: https://reviews.apache.org/r/67006/diff/2/ Changes: https://reviews.apache.org/r/67006/diff/1-2/ Testing ------- As there is not functional change. I juat made sure that all the existing tests pass. Thanks, kalyan kumar kalvagadda