[Koha-bugs] [Bug 15262] Run Koha Run. Koha search should be fast again.

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Nov 30 15:46:02 CET 2015


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15262

--- Comment #6 from Mirko Tietgen <mirko at abunchofthings.net> ---
In the long run, we probably should change things more deeply. Related part of
a dicussion from IRC:

(13:51:58) tcohen: drojf: my plan with
https://github.com/tomascohen/koha/tree/xslt_filtering was to avoid useless
filtering of the retrieved records
(13:52:14) tcohen: so, put search results in Koha::MetadataRecord object
(13:52:42) tcohen: and pass them through a pipe of Koha::Filter-s using the
Koha::RecordProcessor thing
(13:53:09) tcohen: that's from a perspective in which we re-organize the code
(13:53:29) tcohen: and start abstracting things a bit for a future
multi-metadata schema support
(13:54:10) tcohen: regarding zebra, in 3.16 we made DOM setups exchange XML
between Perl and Zebra
(13:54:15) tcohen: both for indexing (which is OK)
(13:54:34) tcohen: and for retrieval (which is OK, but seems really under
performant in the current state of the way we do things)
(13:55:20) tcohen: so my opinion regarding the syspref retrieval for each
record is that we should optimize that in a short term
(13:55:35) tcohen: but move to Koha::Fitler ofr the mid/long-term
(13:55:58) tcohen: if you think about it, we are serializing/deserializing
things more than 3 times
(13:56:17) tcohen: and this might get worse with JSON data from ES
(13:56:36) tcohen: get JSON data => have filters for JSON data => period
(13:56:39) drojf: that sounds very good to me
(13:56:51) tcohen: get MARC(XML) data => have filters for MARC(XML) data =>
period
(13:57:26) tcohen: also, with the current indexer, we shouldn't re-calculate
loan status, etc for rendering
(13:57:33) tcohen: that information is already in zebra
(13:57:35) tcohen: or it should
(13:58:13) tcohen: it is cheaper to prepare the data so zebra has it, than
calculate it on rendering
(13:58:31) tcohen: because de indexing process is asynchronous 
(13:58:40) tcohen: and is queued, etc

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list