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.
Created attachment 5566 [details] [review] 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.
Created attachment 5567 [details] [review] 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)
Nice change, pushed please test
Fix is now committed, marking as resolved.