Metarecord summary can be very slow

Bug #2051708 reported by Mike Rylander
30
This bug affects 7 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
High
Unassigned
3.11
Fix Released
High
Unassigned

Bug Description

 * Evergreen version: 3.11+

As part of the angularization of the staff catalog, new logic was added to the bib summary API that gathers record attributes for the constituent records in a result's metarecord. For very large metarecords (~100+ constituents) this can be very slow, and when a search returns many bibs from the same metarecord, the underlying data is retrieved separately for each bib.

There are certainly some optimizations that could be applied to the MR summary calculation, but as an immediate measure, the MR summary should be cached within page-level API call so that each MR summary for a result page only needs to be calculated (and the 5+ second price paid) once per screen.

Branch coming soon...

Revision history for this message
Mike Rylander (mrylander) wrote :

Since this directly addresses the same code touched by the proposed fix for bug #2039229, I will build the fix atop Jeff's branch from that bug.

Changed in evergreen:
milestone: none → 3.11.3
Galen Charlton (gmc)
Changed in evergreen:
importance: Medium → High
Revision history for this message
Mike Rylander (mrylander) wrote :
tags: added: pullrequest
Galen Charlton (gmc)
Changed in evergreen:
assignee: Mike Rylander (mrylander) → Galen Charlton (gmc)
milestone: 3.11.3 → 3.12.1
Revision history for this message
Galen Charlton (gmc) wrote :

Tested and signed off; branch is user/gmcharlt/lp2051708_signoff

This one is a bit urgent for us, so I'll open another bug for further follow-up.

tags: added: signedoff
Changed in evergreen:
assignee: Galen Charlton (gmc) → nobody
Jason Boyer (jboyer)
Changed in evergreen:
assignee: nobody → Jason Boyer (jboyer)
Revision history for this message
Jason Boyer (jboyer) wrote :

Pushed to main, rel_3_12, and rel_3_11. Thanks Mike and Galen!

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Jason Boyer (jboyer) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.