Bug 5064 - Poor response time on acquisitions screen
Summary: Poor response time on acquisitions screen
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: rel_3_2
Hardware: All All
: PATCH-Sent (DO NOT USE) major (vote)
Assignee: Colin Campbell
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-27 12:01 UTC by Colin Campbell
Modified: 2012-10-25 23:03 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Proposed Patch (7.10 KB, patch)
2010-07-27 12:20 UTC, Colin Campbell
Details | Diff | Splinter Review
Revised patch (7.10 KB, patch)
2011-03-08 14:09 UTC, Colin Campbell
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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