webstaff MARC editor can spam requests for fixed field metadata

Bug #1765444 reported by Galen Charlton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.0
Fix Released
Undecided
Unassigned

Bug Description

The web staff MARC editor can issue repeated (and redundant) calls of open-ils.cat.biblio.fixed_field_values.by_rec_type and open-ils.fielder.cmfpm.atomic when initializing itself.

In particular, each eg-marc-edit-fixed-field directive invokes the tag table service to fetch fixed field position and value data. While the results are cached in Hatch, if the results are not /already/ cached, currently each eg-marc-edit-fixed-field directive (of which there are 64) invokes the service. Most of them get a cache miss and consequently make the API calls.

This has been observed to make open-ils.cat services in particular hit configured drone limits.

Evergreen 3.0 and master

Revision history for this message
Galen Charlton (gmc) wrote :
tags: added: cataloging pullrequest webstaffclient
Revision history for this message
Mike Rylander (mrylander) wrote :
Revision history for this message
Cesar V (cesardv) wrote :
Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

We've applied this in our production 3.1 environment. Prior to the fix, we were seeing big spikes in open-ils.cat drone counts; that problem seems to have gone away once the fix was in place.

Changed in evergreen:
milestone: none → 3.2-beta
status: New → Confirmed
Revision history for this message
Mike Rylander (mrylander) wrote :

Merged to master through 3.0. Thanks, Galen, Cesar, and Jeff!

Changed in evergreen:
status: Confirmed → Fix Committed
Changed in evergreen:
milestone: 3.2-beta → 3.1.2
no longer affects: evergreen/3.1
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.