Page MenuHomePhabricator

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

Description

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

Event Timeline

Mattflaschen-WMF raised the priority of this task from to Needs Triage.
Mattflaschen-WMF updated the task description. (Show Details)
Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptDec 1 2015, 7:21 PM
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
Catrope triaged this task as Normal priority.Dec 19 2015, 1:26 AM
Krinkle removed a subscriber: Krinkle.Apr 7 2016, 11:39 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.