----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48102/#review135900 -----------------------------------------------------------
Ship it! Ship It! - Velmurugan Periasamy On May 31, 2016, 11:35 p.m., Sailaja Polavarapu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48102/ > ----------------------------------------------------------- > > (Updated May 31, 2016, 11:35 p.m.) > > > Review request for ranger, Alok Lal, Don Bosco Durai, Abhay Kulkarni, Madhan > Neethiraj, Ramesh Mani, and Velmurugan Periasamy. > > > Bugs: RANGER-1008 > https://issues.apache.org/jira/browse/RANGER-1008 > > > Repository: ranger > > > Description > ------- > > Added code to catch exceptions while performing ldap search and log the > exception so that usersync can complete with rest of the sync process for > both users & groups. > > > Diffs > ----- > > > ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapUserGroupBuilder.java > 0b76883 > ugsync/src/test/java/org/apache/ranger/usergroupsync/LdapUserGroupTest.java > 4355c4d > > Diff: https://reviews.apache.org/r/48102/diff/ > > > Testing > ------- > > 1. Added unit tests to cover the failure cases. > 2. Performed functional tests against openldap & AD servers including both > user based and group based search cases. > > > Thanks, > > Sailaja Polavarapu > >
