https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17250 --- Comment #18 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Jonathan Druart from comment #17)
Another side effect of those code changes is that search speed is now considerably slower, after converting it to DBIx, ->search_by_koha_field followed by ->count and|or ->next is taking ca 8-14 miliseconds; for a search with 50 results (and 2 items per biblio record on average), search performance penalty is somewhere around:
0.012 * 50 * 2 * 3 = 3.6 seconds
(= ~15-25% of the CPU cycles involved in the search are spent in those 3 code parts, while in pre-DBIx version, it was 3-5% of the cycles).
I expected a speed difference, but not so big. The idea behind the AV refactoring is to cache everything to speed up this area. I will need to focus on that.
A quick benchmark using firebug: I notice 1.7s to 2.1s between 1e17c9290 and master. Why do you think ->search_by_koha_field is the guilty? It seems only called one from searchResults (for a search) -- You are receiving this mail because: You are watching all bug changes.