Page MenuHomePhabricator

Move callers away from getMainObjectStash() that do not need it
Closed, ResolvedPublic

Description

Move callers to "db-replicated" or the WAN cache that can avoid using $wgMainStash.

Only non-sessions things that need very fast and DC-replicated stores should keep using it for now.

Details

SubjectRepoBranchLines +/-
mediawiki/extensions/AbuseFiltermaster+206 -209
mediawiki/coremaster+41 -53
mediawiki/extensions/LoginNotifymaster+48 -48
mediawiki/extensions/LoginNotifymaster+2 -1
mediawiki/extensions/FlaggedRevsmaster+76 -68
mediawiki/extensions/FlaggedRevsmaster+90 -79
mediawiki/extensions/Graphmaster+46 -59
mediawiki/extensions/AbuseFiltermaster+65 -34
mediawiki/extensions/FileImportermaster+2 -3
mediawiki/extensions/ConfirmEditmaster+3 -3
mediawiki/coremaster+42 -172
mediawiki/coremaster+58 -64
mediawiki/coremaster+9 -6
mediawiki/coremaster+69 -17
mediawiki/extensions/TorBlockmaster+47 -77
Show related patches Customize query in gerrit

Event Timeline

Change 520964 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/core@master] jobqueue: migrate root job deduplication to the WAN cache

https://gerrit.wikimedia.org/r/520964

Change 520965 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/core@master] upload: move UploadBase status store from $wgMainStash to "db-replicated"

https://gerrit.wikimedia.org/r/520965

Change 520969 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/core@master] Move the showMissingArticle() cache from $wgMainStash to "db-replicated"

https://gerrit.wikimedia.org/r/520969

Change 521014 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/core@master] Remove $wgSiteStatsAsyncFactor feature and related $wgMainStash use

https://gerrit.wikimedia.org/r/521014

Change 521017 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/LoginNotify@master] Use $wgSessionCacheType instead of $wgMainStash

https://gerrit.wikimedia.org/r/521017

Change 521018 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/AbuseFilter@master] Move "block-autopromote" key from $wgMainStash to db-replicated

https://gerrit.wikimedia.org/r/521018

Change 521019 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/ConfirmEdit@master] Migrate captcha sessions from $wgMainStash to $wgSessionCacheType

https://gerrit.wikimedia.org/r/521019

Change 521020 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/TorBlock@master] Convert getExitNodes() from $wgMainStash to the WAN cache

https://gerrit.wikimedia.org/r/521020

Change 521209 had a related patch set uploaded (by Krinkle; owner: Aaron Schulz):
[mediawiki/extensions/WikimediaEvents@master] Move user salt hash storage from $wgMainStash to $wgSessionCacheType

https://gerrit.wikimedia.org/r/521209

Change 521020 merged by jenkins-bot:
[mediawiki/extensions/TorBlock@master] Convert getExitNodes() from $wgMainStash to the WAN cache

https://gerrit.wikimedia.org/r/521020

Change 499969 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/core@master] Use the main stash for basic user talk page notifications

https://gerrit.wikimedia.org/r/499969

Change 522590 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/FlaggedRevs@master] Convert FlaggedRevsStats and ValidationStatistic to using WAN cache

https://gerrit.wikimedia.org/r/522590

Change 522591 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/FlaggedRevs@master] Migrate FRUserActivity store to WAN cache and "db-replicated"

https://gerrit.wikimedia.org/r/522591

Change 522619 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/AbuseFilter@master] Move filter profiling to "db-replicated" and update via AbuseFilterProfileUpdateJob

https://gerrit.wikimedia.org/r/522619

Change 520965 merged by jenkins-bot:
[mediawiki/core@master] upload: move UploadBase status store from $wgMainStash to "db-replicated"

https://gerrit.wikimedia.org/r/520965

Change 520969 merged by jenkins-bot:
[mediawiki/core@master] Move the showMissingArticle() cache from $wgMainStash to "db-replicated"

https://gerrit.wikimedia.org/r/520969

Change 520964 merged by jenkins-bot:
[mediawiki/core@master] jobqueue: migrate root job deduplication to the WAN cache

https://gerrit.wikimedia.org/r/520964

Change 521014 merged by jenkins-bot:
[mediawiki/core@master] Remove $wgSiteStatsAsyncFactor feature and related $wgMainStash use

https://gerrit.wikimedia.org/r/521014

Change 521019 abandoned by Aaron Schulz:
Migrate captcha sessions from $wgMainStash to $wgSessionCacheType

https://gerrit.wikimedia.org/r/521019

Change 524662 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/FileImporter@master] Convert FileImporterSuccessCache to "db-replicated" cache

https://gerrit.wikimedia.org/r/524662

Change 524662 merged by jenkins-bot:
[mediawiki/extensions/FileImporter@master] Convert FileImporterSuccessCache to "db-replicated" cache

https://gerrit.wikimedia.org/r/524662

Krinkle triaged this task as Medium priority.Aug 6 2019, 1:10 AM
Krinkle moved this task from Doing (old) to Blocked (old) on the Performance-Team board.

Change 521018 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Move "block-autopromote" key from $wgMainStash to 'db-replicated'

https://gerrit.wikimedia.org/r/521018

Change 529192 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/LoginNotify@master] [DNM] Switch to "db-replicated" instead of $wgMainStash

https://gerrit.wikimedia.org/r/529192

Change 493628 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/Graph@master] Rely on ParserCache instead of using $wgMainStash in a flakey way

https://gerrit.wikimedia.org/r/493628

Change 493628 merged by jenkins-bot:
[mediawiki/extensions/Graph@master] Rely on ParserCache instead of using $wgMainStash in a flakey way

https://gerrit.wikimedia.org/r/493628

Change 522590 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@master] Convert statistics to WAN cache and local cluster cache locking

https://gerrit.wikimedia.org/r/522590

Change 522591 merged by jenkins-bot:
[mediawiki/extensions/FlaggedRevs@master] Migrate FRUserActivity store to WAN cache and "db-replicated"

https://gerrit.wikimedia.org/r/522591

Change 521017 abandoned by Aaron Schulz:
[DNM] Use 'db-replicated' instead of $wgMainStash

Reason:
duplicate

https://gerrit.wikimedia.org/r/521017

Krinkle subscribed.

Change 522619 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/extensions/AbuseFilter@master] Move filter profiling to "db-replicated" and update via AbuseFilterProfileUpdateJob

https://gerrit.wikimedia.org/r/522619

Tagging CPT for code review on this patch as steward of AbuseFilter. (Looks like Daimona might not have the time or comfort for this refactor).

Grepping for getMainObjectStash() callers, I don't see anything left to do here.

Change 529192 abandoned by Aaron Schulz:

[mediawiki/extensions/LoginNotify@master] [DNM] Switch to "db-replicated" instead of $wgMainStash

Reason:

https://gerrit.wikimedia.org/r/529192

Change 499969 abandoned by Aaron Schulz:

[mediawiki/core@master] Use the main stash for basic user talk page notifications

Reason:

https://gerrit.wikimedia.org/r/499969

Change 522619 abandoned by Aaron Schulz:

[mediawiki/extensions/AbuseFilter@master] Refactor filter profiling and defer updates via AbuseFilterProfileUpdateJob

Reason:

https://gerrit.wikimedia.org/r/522619