Bug 5064

Summary: Poor response time on acquisitions screen
Product: Koha Reporter: Colin Campbell <colin.campbell>
Component: AcquisitionsAssignee: Colin Campbell <colin.campbell>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: major    
Priority: PATCH-Sent (DO NOT USE) CC: chris, julian.maurice, nengard
Version: rel_3_2   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Proposed Patch
Revised patch

Description Colin Campbell 2010-07-27 12:01:30 UTC
Testing 3.2 acq on a site with a large existing acq database, the response time on the initial acq screen was delays of 10 to 12 seconds (on a test system with no load) As this is the initial acq screen it seems unacceptable.

Comparing to the old 3.1 system the number of baskets returned was not consistent.
Comment 1 Colin Campbell 2010-07-27 12:20:12 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2010-12-13 03:15:42 UTC
This patch looks promising, can I get someone to sign off please, then Ill test and push.
Comment 3 Colin Campbell 2010-12-13 12:20:44 UTC
If testing this please also look at the patch for bug 5063 which compliments it
Comment 4 Nicole C. Engard 2011-03-07 18:38:05 UTC
Does not apply:



Apply? [y]es/[n]o/[e]dit/[v]iew patch/[a]ccept all y
Applying: bug_5064 Rework data retrieval in booksellers.pl
fatal: sha1 information is lacking or useless (acqui/booksellers.pl).
Repository lacks necessary blobs to fall back on 3-way merge.
Cannot fall back to three-way merge.
Patch failed at 0001 bug_5064 Rework data retrieval in booksellers.pl
When you have resolved this problem run "git am --resolved".
If you would prefer to skip this patch, instead run "git am --skip".
To restore the original branch and stop patching run "git am --abort".
nengard@debian:~/kohaclone$
Comment 5 Colin Campbell 2011-03-08 14:09:04 UTC
Created attachment 3265 [details] [review]
Revised patch

Patch rebased on current HEAD + bug_5063 patch. It assumes that patch for bug 5063 has been applied first
Comment 6 Chris Cormack 2011-03-11 23:54:00 UTC
Pushed, please test and mark resolved