[Koha-bugs] [Bug 35819] "No job found" error for BatchUpdateBiblioHoldsQueue (race condition)

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

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

--- Comment #40 from Jonathan Druart <jonathan.druart+koha at gmail.com> ---
(In reply to Tomás Cohen Arazi from comment #33)
> Comment on attachment 161527 [details] [review]
> Bug 35819: Improve logging
> 
> Review of attachment 161527 [details] [review]:
> -----------------------------------------------------------------
> 
> ::: misc/workers/background_jobs_worker.pl
> @@ +73,4 @@
> >  $max_processes ||= C4::Context->config('background_jobs_worker')->{max_processes} if C4::Context->config('background_jobs_worker');
> >  $max_processes ||= 1;
> >  
> > +my $not_found_retries = {};
> 
> This will only work for a single worker, right?
> 
> I think it is a good compromise approach, and running multiple workers would
> just make it retry more in the worst case.
> 
> Just noting it.

Yes, this is not ideal at all, but I wanted to avoid the wrong warning in the
log that would cause confusion. As well as preventing an invalid job (should
never happen, right! :D) to block the worker.

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