Too many parallel open-ils.actor.user.itemsout.notices requests

Bug #1913811 reported by Jeff Davis
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
High
Unassigned
3.5
High
Unassigned

Bug Description

EG 3.5

Another follow-up to bug 1896285. The Items Out tab appears to fire off a separate open-ils.actor.user.itemsout.notices call for each row in the grid. If you have enough rows, this can exhaust open-ils.actor drones pretty quickly.

description: updated
tags: added: parallel-requests
Bill Erickson (berick)
Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
status: New → Confirmed
Revision history for this message
Bill Erickson (berick) wrote :
Changed in evergreen:
milestone: none → 3.6.2
tags: added: pullrequest
Changed in evergreen:
assignee: Bill Erickson (berick) → nobody
importance: Undecided → High
Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Thanks, Bill! With the fix in place, loading the Items Out grid no longer causes a spike in open-ils.actor drones. However, for accounts with lots of checkouts, it takes a lot longer to populate the grid -- over 10 seconds for a patron with 100 circs in my test environment. (Without the fix, it starts to populate almost immediately.)

Here's a working branch with a signoff for Bill's commit, plus a second commit adding a progress dialog for the Items Out grid:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1913811-circ-notices-api-parallel-progress-dialog-signoff

Alternatively we could look for a way to load the grid faster while still avoiding excessive API calls.

Revision history for this message
Chris Sharp (chrissharp123) wrote :

Working for me in PINES production. Pushed to master, rel_3_6, and rel_3_5. Thanks, Bill and Jeff!

Changed in evergreen:
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers