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

Search post processing is taking a lot of time > 7000 queries #33554

Closed
AndyScherzinger opened this issue Aug 15, 2022 · 5 comments
Closed

Search post processing is taking a lot of time > 7000 queries #33554

AndyScherzinger opened this issue Aug 15, 2022 · 5 comments
Assignees

Comments

@AndyScherzinger
Copy link
Member

Possible fix: #33550

@PVince81
Copy link
Member

@icewind1991 the PR is merged

can you provide before/after measurements before closing ?

@icewind1991
Copy link
Member

#33550 seems to work as expected

  • 3368 -> 227 for search
  • 3375 -> 528 for recent

Have some ideas for the remaining ones that I'll investigate before closing this

@PVince81
Copy link
Member

@icewind1991 if safe to backport I think we should do it if it makes search much more snappy

@icewind1991
Copy link
Member

I'll trigger the backports

With #33574

  • 50 for search
  • 340 for recent

Recent is mostly larger because of a larger result set, more improvements can probably be done but I'll close this for now.

@icewind1991
Copy link
Member

With https://github.com/nextcloud/server/pull/33575/commits

  • 50 for search
  • 153 for recent

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

3 participants