https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43729
Bug ID: 43729
Summary: Remove STOMP/RabbitMQ background jobs support
Initiative type: ---
Sponsorship ---
status:
Product: Koha
Version: Main
Hardware: All
OS: All
Status: NEW
Severity: enhancement
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: [email protected]
Reporter: [email protected]
QA Contact: [email protected]
Depends on: 42077
Blocks: 41506
Target Milestone: ---
Follow-up to bug 42077 (RabbitMQ made optional, polling the default) and part
of the bug 41506 effort to remove RabbitMQ.
Now that SQL polling is the default and RabbitMQ is no longer a hard
dependency, this bug removes the STOMP/RabbitMQ background-jobs code path
entirely and makes polling the only mechanism.
Scope:
* Koha/BackgroundJob.pm: drop Net::Stomp, the connect() method and the STOMP
enqueue path (send_with_receipt). Keep the enqueue() contract (store
status='new') and process($args).
* Centralize the generic worker loop into a new Koha::Worker class;
misc/workers/background_jobs_worker.pl becomes a thin wrapper
(Koha::Worker->new->run). This supersedes the 'centralize worker code' work on
bug 35920, whose patches will be re-attached here.
* misc/workers/es_indexer_daemon.pl: strip the STOMP branch, keep the
DB-polling batched indexer loop.
* Remove the JobsNotificationMethod system preference: delete from
sysprefs.sql, add an atomicupdate to DELETE it, and remove the entry from
admin.pref.
* about.pl / about.tt: remove the job_notification_method_check()
broker-connection check, the warnConnectBroker warning and the 'Message broker'
row.
* Tests: drop STOMP-specific coverage in t/db_dependent/Koha/BackgroundJob.t
(connect subtest, send_with_receipt mock) and
t/db_dependent/Koha/BackgroundJobs.t.
Out of scope (future): unifying zebraqueue + elastic_index into a shared
indexing_queue and a batched worker abstraction; folding rebuild_zebra.pl -d
into it.
Depends on: bug 42077. See also: bug 35655 (introduced the pref), bug 35920
(worker centralization).
Referenced Bugs:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41506
[Bug 41506] Should we remove RabbitMQ?
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42077
[Bug 42077] Move RabbitMQ from mandatory to optional and make SQL polling the
default
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
