Page MenuHomePhabricator

Make memcached BagOStuff classes use 'gets' only if a CAS token is needed
Closed, ResolvedPublic


This saves a few bytes on the response and, more importantly, makes mcrouter warmup-style config like more practical when using BagOStuff. Most get() operations, and all getMulti() ones, do not need CAS tokens since they are not being used within BagOStuff::merge().

Event Timeline

aaron created this task.Jul 2 2020, 9:29 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 2 2020, 9:29 PM

Change 609269 had a related patch set uploaded (by Aaron Schulz; owner: Aaron Schulz):
[mediawiki/core@master] objectcache: only use memcached "gets" when tokens are needed

aaron moved this task from Inbox to Doing (old) on the Performance-Team board.Jul 6 2020, 8:05 PM

Change 609269 merged by jenkins-bot:
[mediawiki/core@master] objectcache: only use memcached "gets" when tokens are needed

Krinkle closed this task as Resolved.Jul 20 2020, 8:08 PM