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

Solr performance issues #7017

Closed
seabelis opened this issue Sep 27, 2022 · 5 comments
Closed

Solr performance issues #7017

seabelis opened this issue Sep 27, 2022 · 5 comments
Assignees
Labels
Lead: @cdrini Issues overseen by Drini (Staff: Team Lead & Solr, Library Explorer, i18n) [managed] Module: Solr Issues related to the configuration or use of the Solr subsystem. [managed] Priority: 1 Do this week, receiving emails, time sensitive, . [managed] Theme: Performance Issues related to UI or Server performance. [managed] Theme: Search Issues related to search UI and backend. [managed] Type: Bug Something isn't working. [managed]
Milestone

Comments

@seabelis
Copy link
Collaborator

Multiple folks have noted solr performance issues as noted below.

From community call notes...
Autocomplete

  • adding authors
  • Languages
  • Changing work IDs for moving a work
  • Subjects
  • Searching by ISBN; there’s a delay before the preview appears (main nav search box)

Consistent in the past ~month
Might be everything in this autocomplete

Lisa: Searching frequently produces no results but then searching the same terms again produces results
+Onno
Mark: First time I load OL in the day doesn’t work
-Lisa

Evidence / Screenshot (if possible)

Relevant url?

Steps to Reproduce

  1. Go to ...any edit form
  2. Do ...try adding an author or language
  • Actual: delay is longer than expected and longer than it has been in the past
  • Expected:

Details

  • Logged in (Y/N)?
  • Browser type/version?
  • Operating system?
  • Environment (prod/dev/local)? prod

Proposal & Constraints

Related files

Stakeholders

@cdrini

@seabelis seabelis added Type: Bug Something isn't working. [managed] Module: Solr Issues related to the configuration or use of the Solr subsystem. [managed] Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] Needs: Lead labels Sep 27, 2022
@mekarpeles mekarpeles added Theme: Search Issues related to search UI and backend. [managed] Lead: @cdrini Issues overseen by Drini (Staff: Team Lead & Solr, Library Explorer, i18n) [managed] and removed Needs: Lead labels Sep 27, 2022
@mekarpeles
Copy link
Member

Similar to #6377 epic

@tfmorris
Copy link
Contributor

tfmorris commented Sep 30, 2022

Performance is erratic and miserable. I'd suggest adding latency metrics and monitoring them so that you don't have to wait for users to spend a month being pissed off.

Some of the symptoms I've seen include:

  • autocomplete taking several seconds

  • page renders failing with obscure errors (e.g. "plural must be integer, not None") because solr delivered information isn't available
    Screen Shot 2022-09-30 at 4 57 51 PM

  • author merge page rendering with no works for any author (but also no error message)

  • author merge pages rendering with no works for some of the authors - makes me wonder if the missing works will fail to get moved if I don't notice and refresh the page before merging

  • page renders failing with "connection retry count exceeded" trying to connect to solr backend

  • author search page listing authors with none of their works (and a count of 1 - another longstanding bug)

@cdrini
Copy link
Collaborator

cdrini commented Oct 4, 2022

Howdy folks I've patch deployed a number of solr config improvements which should address these last night. Please let me know if things look better or if they do not!

@cdrini cdrini added Priority: 1 Do this week, receiving emails, time sensitive, . [managed] and removed Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] labels Oct 4, 2022
@cdrini cdrini modified the milestones: Active Sprint, Next (proposed) Oct 4, 2022
@cdrini cdrini self-assigned this Oct 4, 2022
@cdrini
Copy link
Collaborator

cdrini commented Oct 10, 2022

Howdy folks, our sentry metrics are looking muuuuch better; way fewer errors!

image

This should address a lot of the "things appearing with 0 results", "got NoneType", etc.

And search load times are stable in our grafana. Closing this for now since it's been a week and things look better.

Does autocomplete perf look better, @seabelis @tfmorris ?

@cdrini cdrini closed this as completed Oct 10, 2022
@tfmorris
Copy link
Contributor

Yes, the site is usable again now. I miss having the updated info available immediately in search after edits, but it's only a minute or two delayed, and better than having the site be unusable.

Since there are apparently metrics in place already (based on the graph above), I'd suggest setting an alarm threshold so that devs get notified when they break something.

@mekarpeles mekarpeles added the Theme: Performance Issues related to UI or Server performance. [managed] label Nov 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Lead: @cdrini Issues overseen by Drini (Staff: Team Lead & Solr, Library Explorer, i18n) [managed] Module: Solr Issues related to the configuration or use of the Solr subsystem. [managed] Priority: 1 Do this week, receiving emails, time sensitive, . [managed] Theme: Performance Issues related to UI or Server performance. [managed] Theme: Search Issues related to search UI and backend. [managed] Type: Bug Something isn't working. [managed]
Projects
None yet
Development

No branches or pull requests

4 participants