RANGER-5764:Enhancing RangerUserStore to include or exclude user group mappings - #1185
Open
abhiishek26 wants to merge 1 commit into
Open
RANGER-5764:Enhancing RangerUserStore to include or exclude user group mappings #1185abhiishek26 wants to merge 1 commit into
abhiishek26 wants to merge 1 commit into
Conversation
Contributor
|
@abhiishek26 , please check if you have used the correct JIRA ID in the commit. |
mneethiraj
reviewed
Aug 25, 2026
| </property> | ||
| <!-- User-store download Properties starts--> | ||
| <property> | ||
| <name>ranger.admin.userstore.download.usergroups.enabled</name> |
Contributor
There was a problem hiding this comment.
I suggest renaming the property to ranger.admin.userstore.include.usergroups.mappings.
Contributor
|
@abhiishek26 , can you also update the title of the commit with correct jira number? (https://issues.apache.org/jira/browse/RANGER-5764) |
abhiishek26
force-pushed
the
UserStore-enhancements
branch
from
August 26, 2026 12:12
5c76aed to
2604bcb
Compare
Contributor
Author
Thanks Vikas , Sorry it was a mismatch |
Contributor
Author
Thanks @spolavarpau1 , I have updated it |
abhiishek26
force-pushed
the
UserStore-enhancements
branch
from
August 26, 2026 12:37
2604bcb to
5bbd0a8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this pull request?
1>Currently, when RangerUserStoreCache refreshes the user store from the database, it always attempts to load user-to-group mappings. In large deployments with a high number of users and groups, loading these mappings on every refresh can significantly increase overhead .
2>I have made it to flag based approach , if flag is set to true then the mappings to the plugins will be provided by the UserStore download apis else not .
3>The major cause of performance bottleneck was final Map<String, Set> userGroups = xUserMgr.getUserGroups();
which has been changed to
final Map<String, Set> userGroups = downloadUserGroupsEnabled ? xUserMgr.getUserGroups() : new HashMap<>();
so based on the requirements we can download or skip user-group mappings in UserStore for Plugins
How was this patch tested?
1>For performance testing this patch was tested for 1 million users and .5 million groups where every 2 user was mapped to one group creating .5 million mappings
2>mvn clean test and mvn clean install were performed locally