From 85b1eb1dc9c6197b10c98ed44c7849aeaa07dc8c Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Tue, 11 Apr 2023 13:43:51 +0000 Subject: [PATCH] Bug 33486: Remove Koha::BackgroundJob::UpdateElasticIndex->process Signed-off-by: Jonathan Druart Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 42c1059d1dd2821e61010f3f6ea0d8d52a699d05) Signed-off-by: Martin Renvoize --- Koha/BackgroundJob/UpdateElasticIndex.pm | 59 +++++------------------- 1 file changed, 11 insertions(+), 48 deletions(-) diff --git a/Koha/BackgroundJob/UpdateElasticIndex.pm b/Koha/BackgroundJob/UpdateElasticIndex.pm index 55c50a1295..1949728077 100644 --- a/Koha/BackgroundJob/UpdateElasticIndex.pm +++ b/Koha/BackgroundJob/UpdateElasticIndex.pm @@ -42,54 +42,17 @@ sub job_type { return 'update_elastic_index'; } -=head3 process - -Process the modification. - -=cut - -sub process { - my ( $self, $args ) = @_; - - $self->start; - - my @record_ids = @{ $args->{record_ids} }; - my $record_server = $args->{record_server}; - - my $report = { - total_records => scalar @record_ids, - total_success => 0, - }; - - my @messages; - eval { - my $es_index = - $record_server eq "authorityserver" - ? $Koha::SearchEngine::AUTHORITIES_INDEX - : $Koha::SearchEngine::BIBLIOS_INDEX; - my $indexer = Koha::SearchEngine::Indexer->new({ index => $es_index }); - $indexer->update_index(\@record_ids); - }; - if ( $@ ) { - warn $@; - push @messages, { - type => 'error', - code => 'index_error', - error => $@, - - } - } else { - $self->step; - # FIXME This is not correct if some record_ids have been skipped - $report->{total_success} = scalar @record_ids; - } - - my $data = $self->decoded_data; - $data->{messages} = \@messages; - $data->{report} = $report; - - $self->finish( $data ); -} +# While most background jobs provide this method, the ES indexing has its own dedicated worker: +# misc/workers/es_index_daemon.pl +# That worker will handle all job processing. +#=head3 process +# +#Process the modification. +# +#=cut +# +#sub process { +#} =head3 enqueue -- 2.39.5