gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 39076: Use state variable to avoid creating extra ES connections 

As elsewhere in the code, we should use a state variable for our
'bib_searcher' object to avoid overhead of extra ES connections.

Worth noting - we don't actually use the 'bib_searcher' at all during
record auto linking as we pass skipmetadata

To test:
0 - Apply patch
1 - Enable biblio auto linking
    AutoLinkBiblios, CatalogModuleRelink, LinkerRelink
2 - Stage and import a marc file
3 - Confirm record authorities link as expected
4 - Search authorities in the staff interface
5 - Confirm usage counts are shown for results

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit e2207cb478)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
Nick Clemens 2025-02-10 13:09:31 +00:00 committed by Fridolin Somers
parent d31f8d7279
commit e2a3710b7d
No known key found for this signature in database
GPG Key ID: 9A8E73E2A8B96537
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Koha/SearchEngine/Elasticsearch/Search.pm
View File

@ -214,7 +214,8 @@ sub search_auth_compat {
my $schema = $database->schema();
my $res = $self->search($query, undef, $count, %options);
my $bib_searcher = Koha::SearchEngine::Elasticsearch::Search->new({index => 'biblios'});
# Use state variables to avoid recreating the objects every time.
state $bib_searcher = Koha::SearchEngine::Elasticsearch::Search->new( { index => 'biblios' } );
my @records;
my $hits = $res->{'hits'};
foreach my $es_record (@{$hits->{'hits'}}) {