http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6078 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|gmcharlt@gmail.com |julian.maurice@biblibre.com --- Comment #1 from Julian Maurice <julian.maurice@biblibre.com> 2011-04-06 08:26:49 UTC --- Created attachment 3724 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=3724 Patch -- 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=6078 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |alex.arnaud@biblibre.com --- Comment #2 from Alex Arnaud <alex.arnaud@biblibre.com> 2011-04-07 14:59:37 UTC --- *** Bug 6123 has been marked as a duplicate of this bug. *** -- 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=6078 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #3724|0 |1 is obsolete| | --- Comment #3 from Chris Cormack <chris@bigballofwax.co.nz> 2011-06-07 06:41:58 UTC --- Created attachment 4390 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=4390 Signed off Patch -- 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=6078 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #4390|0 |1 is obsolete| | --- Comment #4 from Paul Poulain <paul.poulain@biblibre.com> 2011-08-10 08:56:08 UTC --- Created attachment 4917 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=4917 proposed patch (passed QA) QA comment the goal of the patch is correct: nested queries are time consumming. The solution provided requires quite a lot of SQL calculation, but there is probably no best solution, and this one is an improvement Results are the same before and after this patch Marking passed QA -- 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=6078 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Passed QA |Patch Pushed --- Comment #5 from Chris Cormack <chris@bigballofwax.co.nz> 2011-08-10 09:14:38 UTC --- Original query results in an execution plan of +----+--------------------+------------+--------+-------------------------------+-------------------+---------+------+------+--------------------------+ | id | select_type | table | type | possible_keys | key | key_len | ref | rows | Extra | +----+--------------------+------------+--------+-------------------------------+-------------------+---------+------+------+--------------------------+ | 1 | PRIMARY | borrowers | index | NULL | PRIMARY | 4 | NULL | 58 | Using where | | 2 | DEPENDENT SUBQUERY | <derived3> | system | NULL | NULL | NULL | NULL | 1 | | | 3 | DERIVED | old_issues | range | old_issuesborridx,old_bordate | old_issuesborridx | 5 | NULL | 8 | Using where; Using index | | 4 | UNION | issues | index | NULL | issuesborridx | 5 | NULL | 1 | Using index | | NULL | UNION RESULT | <union3,4> | ALL | NULL | NULL | NULL | NULL | NULL | | +----+--------------------+------------+--------+-------------------------------+-------------------+---------+------+------+--------------------------+ New query, is much better +----+--------------------+------------+-------+-------------------------------+-------------------+---------+-------------------------------+------+--------------------------+ | id | select_type | table | type | possible_keys | key | key_len | ref | rows | Extra | +----+--------------------+------------+-------+-------------------------------+-------------------+---------+-------------------------------+------+--------------------------+ | 1 | PRIMARY | borrowers | index | NULL | PRIMARY | 4 | NULL | 58 | Using where | | 3 | DEPENDENT SUBQUERY | old_issues | ref | old_issuesborridx,old_bordate | old_issuesborridx | 5 | koha.borrowers.borrowernumber | 8 | Using where; Using index | | 2 | DEPENDENT SUBQUERY | issues | ref | issuesborridx,bordate | issuesborridx | 5 | koha.borrowers.borrowernumber | 1 | Using where; Using index | +----+--------------------+------------+-------+-------------------------------+-------------------+---------+-------------------------------+------+--------------------------+ Pushing, 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.