MushroomObserver / mushroom-observer / 28967435819

99%

main: 99%

Pull #4727

github

Split Solid Queue workers into default + maintenance queue pools

Part of the crontab-to-Solid-Queue audit (issue #4726). config/queue.yml
was fully commented out (gem defaults: one process, 5 threads, queues:
"*"). Splits that into two worker pools - default (4 threads, for
user-facing jobs like ImageDhashJob/ImageLoaderJob/FieldSlipJob/
InatImportJob/InatImportRecoveryJob) and maintenance (2 threads, for
periodic background housekeeping) - so a multi-minute maintenance job
run can't starve user-facing work of threads, and vice versa.

Both pools run under the same single supervisor process (issue #4639/
PR #4657 established there must be exactly one supervisor in
production) - this only splits its worker threads by queue.

Moves the two existing pure-housekeeping recurring jobs
(DiscardStaleJobsJob, UpdateBoxAreaAndCenterColumnsJob) onto the new
maintenance queue. InatImportRecoveryJob stays on default - despite
running on a recurring schedule, it finalizes user-visible import
status (tells the user their import crashed/needs restarting), so it's
user-facing work, not background housekeeping.

The 6 new recurring jobs from issue #4726's Bucket A will land on
:maintenance too, in their own follow-up PRs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

2 of 2 new or added lines in 2 files covered. (100.0%)

51712 of 52180 relevant lines covered (99.1%)

750.88 hits per line