Page MenuHomePhabricator

Add debug/warn for cache reads from in-process Memcached cache wrapper
Open, MediumPublic

Description

Instrument these so we can see where they're occurring and if they're necessary.

Event Timeline

Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald Transcript
Mattflaschen-WMF renamed this task from Add debug/warn for cache reads from in-process cache to Add debug/warn for cache reads from in-process Memcached cache wrapper.Dec 1 2015, 7:24 PM

@Mattflaschen-WMF @SBisson Do we still need to do this, with Stephane's current Flow caching patch?

The issue this is trying to prevent (reading from the cache on POST) is still a possible problem. It would either return null/empty, or trigger a cache population from master, neither of which are intended.

However, in practice I haven't yet hit any weirdness with the patch.