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

(Updated March 10, 2017, 8:56 p.m.)


Review request for ranger, Ankita Sinha, Gautam Borad, Abhay Kulkarni, Madhan 
Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan 
Periasamy.


Changes
-------

After fixing some minor issue of using the new filenames in the test code, 
regenrated the full patch.


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


Repository: ranger


Description
-------

Made code changes to read both password file and group file from the usersync 
configuration file instead of using hardcoded values. Also added/modified 
corresponding unit tests.


Diffs (updated)
-----

  
ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java
 2c9365d 
  
ugsync/src/main/java/org/apache/ranger/unixusersync/process/UnixUserGroupBuilder.java
 f5a4c9a 
  
ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestUnixUserGroupBuilder.java
 831e92d 
  ugsync/src/test/resources/groupFile.txt PRE-CREATION 
  ugsync/src/test/resources/passwordFile.txt PRE-CREATION 


Diff: https://reviews.apache.org/r/57488/diff/3/

Changes: https://reviews.apache.org/r/57488/diff/2-3/


Testing
-------

1. Successfully ran unit tests.
2. Also verified new & existing functionality of unix sync on cluster with 
replacing the unixusersync jar file containing the changes.


Thanks,

Sailaja Polavarapu

Reply via email to