Re: Review Request 73685: RANGER-3505: Ranger usersync fails to sync users when a duplicate user exists in ranger

2021-11-10 Thread Pradeep Agrawal

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


Ship it!




Ship It!

- Pradeep Agrawal


On Nov. 3, 2021, 10:59 p.m., Sailaja Polavarapu wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73685/
> ---
> 
> (Updated Nov. 3, 2021, 10:59 p.m.)
> 
> 
> Review request for ranger, Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, 
> Ramesh Mani, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3505
> https://issues.apache.org/jira/browse/RANGER-3505
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> modified code to ignore case while validating a user for update
> 
> 
> Diffs
> -
> 
>   security-admin/src/main/java/org/apache/ranger/service/XUserService.java 
> f22bcec75 
> 
> 
> Diff: https://reviews.apache.org/r/73685/diff/1/
> 
> 
> Testing
> ---
> 
> Patched cluster with Mysql DB as backed for ranger and verified few test 
> cases.
> Also verified with postgres DB as backend for ranger for quick regression 
> tests.
> 
> 
> Thanks,
> 
> Sailaja Polavarapu
> 
>



Review Request 73685: RANGER-3505: Ranger usersync fails to sync users when a duplicate user exists in ranger

2021-11-03 Thread Sailaja Polavarapu

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

Review request for ranger, Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, 
Ramesh Mani, and Velmurugan Periasamy.


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


Repository: ranger


Description
---

modified code to ignore case while validating a user for update


Diffs
-

  security-admin/src/main/java/org/apache/ranger/service/XUserService.java 
f22bcec75 


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


Testing
---

Patched cluster with Mysql DB as backed for ranger and verified few test cases.
Also verified with postgres DB as backend for ranger for quick regression tests.


Thanks,

Sailaja Polavarapu