----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73303/#review222914 -----------------------------------------------------------
ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/XUserInfo.java Line 112 (original), 113 (patched) <https://reviews.apache.org/r/73303/#comment312086> please consider this, as this this.otherAttrsMap already initialized. if (otherAttrsMap != null) { this.otherAttrsMap = otherAttrsMap; } if this.otherAttrsMap can be null then you can have it as this.otherAttrsMap = otherAttrsMap - Ramesh Mani On April 27, 2021, 10:42 p.m., Sailaja Polavarapu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73303/ > ----------------------------------------------------------- > > (Updated April 27, 2021, 10:42 p.m.) > > > Review request for ranger, Abhay Kulkarni, Mehul Parikh, Ramesh Mani, and > Velmurugan Periasamy. > > > Bugs: RANGER-3254 > https://issues.apache.org/jira/browse/RANGER-3254 > > > Repository: ranger > > > Description > ------- > > Added additional checks before updating user or group attributes to ranger. > Also fixed issue for not updating group memberships if same user with a > different sync source or DN already exists in Ranger. > > > Diffs > ----- > > > ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/XGroupInfo.java > 5f5c9aa38 > ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/XUserInfo.java > 058b98467 > > ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java > 4d8a32a8a > > > Diff: https://reviews.apache.org/r/73303/diff/2/ > > > Testing > ------- > > 1. Verified all the existing unit tests pass. > 2. Patched cluster and verified functionality with syncing users from unix as > well as from AD. > > > Thanks, > > Sailaja Polavarapu > >
