[Bug 33942] New: Improve performance of local holds priority by temporarily caching patrons and items
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33942 Bug ID: 33942 Summary: Improve performance of local holds priority by temporarily caching patrons and items Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Hold requests Assignee: koha-bugs@lists.koha-community.org Reporter: kyle@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com For records with many item level holds by many patrons ( with multiple item level holds per record ), we fetch the item and patron from the database fresh on each loop iteration. If we store these in a hash it should speed this up in the case of large numbers of these holds. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33942 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |kyle@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33942 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33942 --- Comment #1 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 152122 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=152122&action=edit Bug 33942: Improve performance of local holds priority by temporarily caching patrons and items Test Plan: 1) Apply this patch 2) prove -r t/db_dependent/Hold* 3) Note all tests continue to pass -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33942 Sam Lau <samalau@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |samalau@gmail.com Status|Needs Signoff |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33942 --- Comment #2 from Sam Lau <samalau@gmail.com> --- Using index info to reconstruct a base tree... M C4/Reserves.pm Falling back to patching base and 3-way merge... Auto-merging C4/Reserves.pm CONFLICT (content): Merge conflict in C4/Reserves.pm error: Failed to merge in the changes. Patch failed at 0001 Bug 33942: Improve performance of local holds priority by temporarily caching patrons and items -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33942 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com Status|Patch doesn't apply |RESOLVED Resolution|--- |WONTFIX --- Comment #3 from Nick Clemens <nick@bywatersolutions.com> --- This seems to only offer benefit in a small number of edge cases, we won't pursue this one -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org