Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optimize user count retrieval logic in user filter operation to reduce extensive DB calls #21364

Closed
bhagyasakalanka opened this issue Oct 15, 2024 · 1 comment

Comments

@bhagyasakalanka
Copy link
Contributor

Is your suggestion related to an experience ? Please describe.
During user listing, we observed inefficiencies with numerous extensive database calls. For instance, to retrieve the total user count, all user records are fetched instead of just the count. Additionally, there are multiple duplicate calls contributing to the overhead.

Describe the improvement
A new method to retrieve the total user count was introduced to the kernel with PR [1]. However, due to the added complexities in the SCIM2 component, this method cannot be directly applied at the SCIM layer.

To address this, we have raised PR [2] to extend the kernel’s capability to the SCIM layer. However, because of the complexities involved, directly integrating this method would disrupt existing behavior. As a result, this improvement needs to be considered carefully to ensure smooth implementation without breaking existing functionality at the SCIM level.

[1] wso2/carbon-kernel#4033
[2] wso2-extensions/identity-inbound-provisioning-scim2#562

@bhagyasakalanka
Copy link
Contributor Author

This is complete with: wso2-extensions/identity-inbound-provisioning-scim2#579

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant