for the job queue use case we have a lot of very low-volume job types and it doesn't make sense to designate a special worker for all of them, so we need to create a catch-all-others type of rule in ChangeProp - it will look at other rules and subscribe to all topics matching a pattern that other rules are not subscribed to yet.
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Separate regexed rules for multi-topic rules support. | mediawiki/services/change-propagation/deploy | master | +15 -3 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | • Pchelolo | T157088 [EPIC] Develop a JobQueue backend based on EventBus | |||
| Resolved | • Pchelolo | T190327 FY17/18 Q4 Program 8 Services Goal: Complete the JobQueue transition to EventBus | |||
| Resolved | • Pchelolo | T191238 Add support for catch-all rule in ChangeProp |
Event Timeline
Comment Actions
Mentioned in SAL (#wikimedia-operations) [2018-04-04T16:47:02Z] <ppchelko@tin> Started deploy [cpjobqueue/deploy@d4a84ae]: Support multi-topic rules T191238
Comment Actions
Mentioned in SAL (#wikimedia-operations) [2018-04-04T16:47:44Z] <ppchelko@tin> Finished deploy [cpjobqueue/deploy@d4a84ae]: Support multi-topic rules T191238 (duration: 00m 42s)
Comment Actions
The multi-topic rule support was added to change-prop ad deployed for the JobQueue.
What's left is to make that exclusive so that we don't double-subscribe if we have a regex catch-all rule
Comment Actions
Change 424337 had a related patch set uploaded (by Ppchelko; owner: Ppchelko):
[mediawiki/services/change-propagation/deploy@master] Separate regexed rules for multi-topic rules support.
Comment Actions
Change 424337 merged by Mobrovac:
[mediawiki/services/change-propagation/deploy@master] Separate regexed rules for multi-topic rules support.
Comment Actions
The feature was merged and released for the JobQueue, the related config changes for ChangeProp was also merged, resolving.