https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38646

--- Comment #12 from David Cook <[email protected]> ---
(In reply to David Cook from comment #11)
> Another thing to note... getLanguages() is called both when loading the
> advanced search and when running a search and getting the results, but the
> output of this function is only ever used by the advanced search.

(In reply to Martin Renvoize (ashimema) from comment #5)
> I'd love to see a benchmark too.. failing QA for now for the failing test.

So I just did a patch... and the performance difference when running
catalogue/search.pl with getLanguages() and without getLanguages() is
negligible in koha-testing-docker. 

I had runs where it was slower running without getLanguages(), so I can't
attribute the presence/absence of getLanguages() to any speed differences.

That said... I think it's a good idea to reduce our volume of DB calls wherever
possible, and maybe with a Koha DB under load you'd see a noticeable speed
difference.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to