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

Performance issues #377

Closed
mejo- opened this issue Aug 16, 2022 · 0 comments · Fixed by #445
Closed

Performance issues #377

mejo- opened this issue Aug 16, 2022 · 0 comments · Fixed by #445

Comments

@mejo-
Copy link
Member

mejo- commented Aug 16, 2022

In GitLab by @ognarb on Aug 16, 2022, 12:06

I profiled a bit collectives on our perftesting instance and /index.php/apps/collectives/_api/12/_pages call is quite bad. One of the reasons for this is that the parent are not cached in PageService::getParentPageId. This then created 900 DB requests

One idea how to fix it is to do something similar to nextcloud/end_to_end_encryption#316 (who was also affected by the same bug) or maybe even try to finish nextcloud/server#31793 so that apps don't have to the caching themselves and duplicate the caching (this is also quite a bit more risky)

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

Successfully merging a pull request may close this issue.

2 participants