Page MenuHomePhabricator

Don't blindly delete and reinsert all settings on save
Closed, ResolvedPublic3 Estimated Story Points


Right now, on global preference save, we delete all the preferences known to the local wiki and reinsert whole dataset, even if only 1 row has changed. Not only this results in increased DB load, it will also cause deadlocks with sufficient concurrency. Just save only the settings that have changed, like most other places we do similar things (e.g. LinksUpdate in core).

Event Timeline

Seems like potentially a lot of work. We should discuss if this is a blocker for releasing to all Wikipedias.

TBolliger set the point value for this task to 3.Jun 5 2018, 11:39 PM
TBolliger moved this task from To Be Estimated/Discussed to Estimated on the Community-Tech board.

Change 440046 had a related patch set uploaded (by MaxSem; owner: MaxSem):
[mediawiki/extensions/GlobalPreferences@master] Make only needed DB operations when saving preferences

Change 440046 merged by jenkins-bot:
[mediawiki/extensions/GlobalPreferences@master] Make only needed DB operations when saving preferences