-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31848/
-----------------------------------------------------------

Review request for ranger and Selvamohan Neethiraj.


Bugs: RANGER-295
    https://issues.apache.org/jira/browse/RANGER-295


Repository: ranger


Description
-------

RANGER-295: Updated Ranger HDFS plugin for following changes in 
FSPermissionChecker:
- class member 'ugi' is now replaced with 'user' and 'groups'
- signature of checkPermissions() method is now changed, as given below:
   New signature: void checkPermission(INodesInPath inodesInPath, boolean 
doCheckOwner, FsAction ancestorAccess, FsAction parentAccess, FsAction access, 
FsAction subAccess, boolean ignoreEmptyDir)
   Old signature: void checkPermission(String path, FSDirectory dir, boolean 
doCheckOwner, FsAction ancestorAccess, FsAction parentAccess, FsAction access, 
FsAction subAccess, boolean ignoreEmptyDir, boolean resolveLink)

Updated HDFS plugin will work with both new version of FSPermissionChecker and 
earlier version of FSPermissionChecker.


Diffs
-----

  
hdfs-agent/src/main/java/org/apache/hadoop/hdfs/server/namenode/RangerFSPermissionChecker.java
 151360f 
  
hdfs-agent/src/main/java/org/apache/ranger/authorization/hadoop/agent/HadoopAuthClassTransformer.java
 1f21053 
  
hdfs-agent/src/test/java/org/apache/ranger/authorization/hadoop/agent/TestRangerFSPermissionChecker.java
 1405bf3 

Diff: https://reviews.apache.org/r/31848/diff/


Testing
-------

Verified that the plugin works with HDFS with updated FSPermissionChecker and 
its earlier version.


Thanks,

Madhan Neethiraj

Reply via email to