Skip to content

Add cache for application privileges (#55836)#58798

Merged
ywangd merged 7 commits intoelastic:7.xfrom
ywangd:es-54317-app-privilege-cache-7.x
Jul 2, 2020
Merged

Add cache for application privileges (#55836)#58798
ywangd merged 7 commits intoelastic:7.xfrom
ywangd:es-54317-app-privilege-cache-7.x

Conversation

@ywangd
Copy link
Copy Markdown
Member

@ywangd ywangd commented Jul 1, 2020

Add caching support for application privileges to reduce number of round-trips to security index when building application privilege descriptors.

Privilege retrieving in NativePrivilegeStore is changed to always fetching all privilege documents for a given application. The caching is applied to all places including "get privilege", "has privileges" APIs and CompositeRolesStore (for authentication).

ywangd added 2 commits July 1, 2020 15:59
Add caching support for application privileges to reduce number of round-trips to security index when building application privilege descriptors.

Privilege retrieving in NativePrivilegeStore is changed to always fetching all privilege documents for a given application. The caching is applied to all places including "get privilege", "has privileges" APIs and CompositeRolesStore (for authentication).
@ywangd ywangd added the backport label Jul 1, 2020
@ywangd
Copy link
Copy Markdown
Member Author

ywangd commented Jul 1, 2020

@elasticmachine test this please

1 similar comment
@ywangd
Copy link
Copy Markdown
Member Author

ywangd commented Jul 1, 2020

@elasticmachine test this please

@ywangd ywangd merged commit a5a8b4a into elastic:7.x Jul 2, 2020
ywangd added a commit that referenced this pull request Jul 3, 2020
The version needs to be adjusted to v7.9.0 after backport #58798
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant