Page MenuHomePhabricator

CacheAwarePropertyInfoStore & CachingPropertyInfoLookup should use WANObjectCache instead of BagOStuff
Closed, ResolvedPublic5 Estimated Story Points


Per T97368#5018549

When writing a combination of delete and set should be used per T218197#5021050

Event Timeline

Addshore created this task.
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Adding also @aaron that knows better what is best (and hopefully can confirm what I am saying).

Yes, and CacheAwarePropertyInfoStore should use delete() or such for purges rather than set(). Using set() would only effect one DC.

Addshore set the point value for this task to 5.Mar 19 2019, 3:41 PM

Change 498348 had a related patch set uploaded (by Addshore; owner: Addshore):
[mediawiki/extensions/Wikibase@master] WIP DNM WANCACHE for property info store?

Change 498348 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Use WanCache for PropertyInfoStore things

We should watch and see some significant improvements .. if none, maybe this wasn't enough. I'm watching it for a week or two

It looks like the train didn't make it past group0 this week, so no change really spotted yet.