[Koha-bugs] [Bug 9988] Leave larger authority merges to merge_authorities cronjob ( pref AuthorityMergeLimit)

[bugzilla-daemon at bugs.koha-community.org]

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

--- Comment #106 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
(In reply to Julian Maurice from comment #102)
> 4) Koha::Authorities::get_usage_count and
> Koha::Authorities::linked_biblionumbers look very similar. Is there a real
> performance gain to use get_usage_count instead of linked_biblionumbers ?
> C4::Authorities::merge call them both and it looks like we could save one
> search request by calling only linked_biblionumbers

Yes, there is a performance gain depending on the number of linked records. A
significant difference between both subs is the second and third parameter of
the simple_search_compat call. And the biblionumber extraction of course.
I first worked with one sub doing both, but in the end I felt that separating
them resulted in cleaner code. I would prefer not to change them on this report
anymore.

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