Am 02.10.2014 17:55, schrieb Jeroen De Dauw:
Hey,
We use two CachingEntityRevisionLookup nested into each other: the outer-most
uses a HashBagOStuff to implement in-process caching, the second level uses
memcached.
It is odd to have two different decorator instances for caching around th
EntityRevisionLookup. I suggest to have only a single decorator for caching,
which writes to a caching interface. Then this caching interface can have an
implementation what uses multiple caches, and perhaps have a decorator on that
level.
Went that way first. Didn't work out nicely. I forget why exactly.
I don't care much either way.
--
Daniel Kinzler
Senior Software Developer
Wikimedia Deutschland
Gesellschaft zur Förderung Freien Wissens e.V.