> On March 22, 2017, 11:35 a.m., Colm O hEigeartaigh wrote: > > Looks fine, but change the comment from "create group if not exists" to > > "create group if it does not exist".
Ok. You are right. I modified and updated the patch. Thanks. - Qiang ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57827/#review169689 ----------------------------------------------------------- On March 22, 2017, 12:24 p.m., Qiang Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57827/ > ----------------------------------------------------------- > > (Updated March 22, 2017, 12:24 p.m.) > > > Review request for ranger, Alok Lal, Ankita Sinha, Don Bosco Durai, Colm O > hEigeartaigh, Gautam Borad, Madhan Neethiraj, Ramesh Mani, Selvamohan > Neethiraj, and Velmurugan Periasamy. > > > Bugs: RANGER-1469 > https://issues.apache.org/jira/browse/RANGER-1469 > > > Repository: ranger > > > Description > ------- > > The user group would be duplicate created if the user group existed during > installing Ranger Policy Admin. > > > Diffs > ----- > > security-admin/scripts/setup.sh bc94874 > > > Diff: https://reviews.apache.org/r/57827/diff/2/ > > > Testing > ------- > > > Thanks, > > Qiang Zhang > >
