-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/75077/
-----------------------------------------------------------
(Updated July 5, 2024, 5:14 a.m.)
Review request for ranger, Asit Vadhavkar, Madhan Neethiraj, Monika Kachhadiya,
Siddhesh Phatak, and Subhrat Chaudhary.
Bugs: RANGER-4841
https://issues.apache.org/jira/browse/RANGER-4841
Repository: ranger
Description
-------
RANGER-4841 : Optimising POST xusers/ugsync/groupusers api, by removing the
update user-group membership logic. Moving the logic of getting the
addedBy/updatedBy userId to the parent thread.
Removing hardcoded userId of 1 which is used to set the addedBy/updatedBy
userId of user, group and user-group membership and replacing it by the logged
in user id.
Diffs
-----
security-admin/src/main/java/org/apache/ranger/service/XGroupService.java
98ee62612
security-admin/src/main/java/org/apache/ranger/service/XGroupUserService.java
a4d6640a5
security-admin/src/main/java/org/apache/ranger/service/XUserService.java
5053cd6a5
Diff: https://reviews.apache.org/r/75077/diff/1/
Testing
-------
Validated the performace of POST xusers/ugsync/groupusers api with over 3
million records in x_group_users table.
Below is the duration the api took to create 1000 user-group memberships
Before fix - 18 seconds
After fix - 15.5 seconds
Thanks,
Anand Nadar
