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

Reply via email to