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

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


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


Repository: ranger


Description
-------

Good coding practices for usersync, tagsync, ldap tool configuration. Also 
added "commons-logging" dependency for ldapconfigcheck tool as it is required 
for running the tool.


Diffs
-----

  tagsync/src/main/java/org/apache/ranger/tagsync/process/TagSyncConfig.java 
5f6079ed 
  ugsync/ldapconfigchecktool/ldapconfigcheck/pom.xml 38dd4aac 
  ugsync/ldapconfigchecktool/ldapconfigcheck/scripts/run.sh f3f7ac5c 
  
ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/CommandLineOptions.java
 384ca239 
  
ugsync/ldapconfigchecktool/ldapconfigcheck/src/main/java/org/apache/ranger/ldapconfigcheck/LdapConfig.java
 5a90b1cb 
  
ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapPolicyMgrUserGroupBuilder.java
 d428e755 
  
ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java
 e9e356a4 
  
ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java
 b30b051c 


Diff: https://reviews.apache.org/r/66698/diff/1/


Testing
-------

1. Verified usersync configuration in both secure and non-secure clusters.
2. Verified ldapconfigcheck tool to always prompt for sample authentication 
user password.


Thanks,

Sailaja Polavarapu

Reply via email to