[Bug 14476] New: Improving opac-search performances
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 Bug ID: 14476 Summary: Improving opac-search performances Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: julian.fiol@biblibre.com Reporter: julian.fiol@biblibre.com QA Contact: testopia@bugs.koha-community.org Blocks: 7923 Improving opac-search performances by caching the results of C4::XSLT::transformMARCXML4XSLT. This also impact the performances of C4::XSLT::XSLTParse4Display. My tests (with Devel::NYTProf) showed a gain of about 65% on C4::XSLT::transformMARCXML4XSLT and 35% on C4::XSLT::XSLTParse4Display. The gain of performances is increasing with the numbers of results. Examples: -------- 1/ searching : "sinatra", results : 2 gain : C4::XSLT::transformMARCXML4XSLT = 40% C4::XSLT::XSLTParse4Display = 19% 2/ searching : "Victor Hugo", results : 16 gain : C4::XSLT::transformMARCXML4XSLT = 71% C4::XSLT::XSLTParse4Display = 36% 3/ searching : "a", results : 2060 gain : C4::XSLT::transformMARCXML4XSLT = 82% C4::XSLT::XSLTParse4Display = 51% -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 Julian FIOL <julian.fiol@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 --- Comment #1 from Julian FIOL <julian.fiol@biblibre.com> --- Created attachment 40740 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40740&action=edit bug 14476 : Caching C4::XSLT::transformMARCXML4XSLT This patch is improving the opac-search performances by caching the results of C4::XSLT::transformMARCXML4XSLT My tests (with Devel::NYTProf) showed a gain of about 65% on C4::XSLT::transformMARCXML4XSLT and 35% on C4::XSLT::XSLTParse4Display. The gain of performances is increasing with the numbers of results. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 --- Comment #2 from Gaetan Boisson <gaetan.boisson@biblibre.com> --- Created attachment 42179 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42179&action=edit [SIGNED-OFF] bug 14476 : Caching C4::XSLT::transformMARCXML4XSLT This patch is improving the opac-search performances by caching the results of C4::XSLT::transformMARCXML4XSLT My tests (with Devel::NYTProf) showed a gain of about 65% on C4::XSLT::transformMARCXML4XSLT and 35% on C4::XSLT::XSLTParse4Display. The gain of performances is increasing with the numbers of results. Signed-off-by: Gaetan Boisson <gaetan.boisson@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 Gaetan Boisson <gaetan.boisson@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gaetan.boisson@biblibre.com Attachment #40740|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 --- Comment #3 from Gaetan Boisson <gaetan.boisson@biblibre.com> --- Works for me, although the performance gain was not as big as expected. I will test it again on a larger data set. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 Gaetan Boisson <gaetan.boisson@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |In Discussion CC| |tomascohen@gmail.com --- Comment #4 from Tomás Cohen Arazi <tomascohen@gmail.com> --- I have a concern: if we cache the record to be displayed, then the item statuses won't be up-to-date. If the tradeoff is not enough, maybe this is just not worth. Because people expect to see updated information on the records. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|julian.fiol@biblibre.com |julian.maurice@biblibre.com CC| |julian.maurice@biblibre.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14476 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org