Page MenuHomePhabricator

ParserCache should use CachedBagOStuff
Closed, ResolvedPublic

Description

Getting entries from ParserCache is quick, but on every page view we seem to do it at least twice. We need to wrap the provided BagOStuff into the CachedBagOStuff for ParserCache to introduce another layer of caching. The cached bag should be limited to some small number of entries - we don't want or need to support caching in-memory for use-cases which enumerates ParserOutputs.

Event Timeline

Change 646739 had a related patch set uploaded (by Ppchelko; owner: Ppchelko):
[mediawiki/core@master] Make ParserCache use CachedBagOStuff

https://gerrit.wikimedia.org/r/646739

Change 646739 merged by jenkins-bot:
[mediawiki/core@master] Make ParserCache use CachedBagOStuff

https://gerrit.wikimedia.org/r/646739

Change 671357 had a related patch set uploaded (by Krinkle; owner: Krinkle):
[mediawiki/core@master] ParserCache: Instrument CachedBagOStuff to understand dupe fetches

https://gerrit.wikimedia.org/r/671357

Change 671357 merged by jenkins-bot:
[mediawiki/core@master] ParserCache: Instrument CachedBagOStuff to understand dupe fetches

https://gerrit.wikimedia.org/r/671357