Re: Review Request 73897: RANGER-3659: Ranger Admin goes to OOM when usersync is trying to delete existing group mappings from ranger DB

2022-03-10 Thread Abhay Kulkarni

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


Ship it!




Ship It!

- Abhay Kulkarni


On March 10, 2022, 2:49 a.m., Sailaja Polavarapu wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73897/
> ---
> 
> (Updated March 10, 2022, 2:49 a.m.)
> 
> 
> Review request for ranger, Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, 
> Ramesh Mani, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3659
> https://issues.apache.org/jira/browse/RANGER-3659
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Improved create, update, or delete group user mappings API call to reduce 
> memory usage in ranger admin. These APIs are called by usersync for updating 
> group memberships from sync source to ranger admin. Also added code to use 
> common DB transaction framework.
> 
> 
> Diffs
> -
> 
>   security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 8109968e4 
>   
> security-admin/src/main/java/org/apache/ranger/service/XGroupUserService.java 
> 2b863b006 
> 
> 
> Diff: https://reviews.apache.org/r/73897/diff/1/
> 
> 
> Testing
> ---
> 
> Patched cluster and tested with ~200k group mappings in Ranger DB.
> Also verified basic functionality for regression testing.
> 
> 
> Thanks,
> 
> Sailaja Polavarapu
> 
>



Review Request 73897: RANGER-3659: Ranger Admin goes to OOM when usersync is trying to delete existing group mappings from ranger DB

2022-03-09 Thread Sailaja Polavarapu

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

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


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


Repository: ranger


Description
---

Improved create, update, or delete group user mappings API call to reduce 
memory usage in ranger admin. These APIs are called by usersync for updating 
group memberships from sync source to ranger admin. Also added code to use 
common DB transaction framework.


Diffs
-

  security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 8109968e4 
  security-admin/src/main/java/org/apache/ranger/service/XGroupUserService.java 
2b863b006 


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


Testing
---

Patched cluster and tested with ~200k group mappings in Ranger DB.
Also verified basic functionality for regression testing.


Thanks,

Sailaja Polavarapu