[Bug 6773] New: CircAutoCompl uses unlimited query
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6773 Bug #: 6773 Summary: CircAutoCompl uses unlimited query Classification: Unclassified Change sponsored?: --- Product: Koha Version: rel_3_6 Platform: All OS/Version: All Status: NEW Severity: minor Priority: P5 Component: Architecture, internals, and plumbing AssignedTo: ian.walls@bywatersolutions.com ReportedBy: ian.walls@bywatersolutions.com QAContact: koha-bugs@lists.koha-community.org The SQL in circ/ysearch.pl is unlimited as to the number of results it returns, and since it's called for each character typed into the patron search interface, databases with many patrons (100,000+) can experience resource drain. Further, since the interface only returns 10 results at a time, the bulk of the work done by the query goes to waste. The SQL in circ/ysearch.pl should limit to 10 results only, and the system preference text in the staff client should note this limitation, for the administrator's knowledge. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6773 Ian Walls <ian.walls@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |2084 -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6773 --- Comment #1 from Ian Walls <ian.walls@bywatersolutions.com> 2011-09-23 13:45:40 UTC --- Created attachment 5566 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5566 Signed Off Patch Adds LIMIT 10 to circ/ysearch.pl SQL (to prevent server overload on large borrowers tables). 10 is used because there are only 10 results shown in the interface at a time. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6773 Ian Walls <ian.walls@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 |PATCH-Sent Patch Status|--- |Signed Off -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6773 Ian Walls <ian.walls@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #5566|0 |1 is obsolete| | --- Comment #2 from Ian Walls <ian.walls@bywatersolutions.com> 2011-09-23 13:48:53 UTC --- Created attachment 5567 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5567 Signed off Patch (minor revision to text) Adds LIMIT 10 to SQL search, since only 10 results are returned in the interface at the time. (minor language issue in circ.pref... fixed in this update) -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6773 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz Patch Status|Signed Off |Patch Pushed --- Comment #3 from Chris Cormack <chris@bigballofwax.co.nz> 2011-09-25 02:06:00 UTC --- Nice change, pushed please test -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6773 Ian Walls <ian.walls@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #4 from Ian Walls <ian.walls@bywatersolutions.com> 2011-11-15 12:41:07 UTC --- Fix is now committed, marking as resolved. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org