[Koha-bugs] [Bug 32594] Add a dedicated ES indexing background worker

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

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

--- Comment #41 from Jonathan Druart <jonathan.druart+koha at gmail.com> ---
(In reply to Nick Clemens from comment #39)
> Created attachment 148350 [details] [review]
> Bug 32594: POC

I don't think we should loop over the jobs, we are adding additional (almost
useless) processing. Here we need to do things fast, so let's update them all
at once. To prevent other scripts/jobs to use this, we should not provide a
method (that does not feel correct).

Now we need to decide if we allow indexing jobs from another worker. The detail
page of the job is still wrong " / records have successfully been reindexed.
Some errors occurred.". This is because it expects
Koha::BackgroundJob::UpdateElasticIndex->process to have processed the job, and
provided the correct info. IMO we should not provide 2 ways to process the
jobs, only the new worker should do it. So let's remove the process method and
simplify the job detail view.

Sounds like we should move that to a separate bug...

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