This saves a few bytes on the response and, more importantly, makes mcrouter warmup-style config like https://gerrit.wikimedia.org/r/c/operations/puppet/+/594760 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().
Description
Description
Details
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
objectcache: only use memcached "gets" when tokens are needed | mediawiki/core | master | +51 -23 |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | None | T244852 Upgrade and improve our application object caching service (memcached) | |||
Resolved | jijiki | T244340 Reduce read pressure on mc* servers by adding a machine-local Memcached instance (on-host memcached) | |||
Resolved | aaron | T257003 Make memcached BagOStuff classes use 'gets' only if a CAS token is needed |
Event Timeline
Comment Actions
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
Comment Actions
Change 609269 merged by jenkins-bot:
[mediawiki/core@master] objectcache: only use memcached "gets" when tokens are needed