Currently, Cargo updates backlinks for pages referenced in query results directly from the #cargo_query parser function. This effectively causes it to attempt to write to the primary database in case of a parser cache miss, which can be especially problematic if the primary is read-only (T330253). This functionality could instead be moved to MediaWiki's LinksUpdate mechanism, so that the backlinks are updated at the same time as other MediaWiki link tables, avoiding potential DB writes on page views.
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Move backlink updates to LinksUpdate | mediawiki/extensions/Cargo | master | +275 -32 |
Related Objects
Related Objects
- Mentioned Here
- T330253: Error when database is in read-only mode
Event Timeline
Comment Actions
Change 921440 had a related patch set uploaded (by TK-999; author: TK-999):
[mediawiki/extensions/Cargo@master] [WIP] Move backlink updates to LinksUpdate
Comment Actions
To the extent that I understand this patch, it looks great! I know there are some DB issues with the current backlinks implementation, so it would be great to get them fixed. Good luck with this.
Comment Actions
Change 921440 merged by jenkins-bot:
[mediawiki/extensions/Cargo@master] Move backlink updates to LinksUpdate
Comment Actions
Thank you, I forgot to circle back and close this task.
We completed the rollout of this change about two weeks ago, and saw a notable decrease in DB writes performed by Cargo afterwards, which is very nice.
