Change Details

@Ladsgroup said in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GrowthExperiments/+/748827/7#message-1bdd570f88402cd35eec06bb64cf04b740ab558c: > The cache for MentorPageMentorManager is ObjectCache::getLocalClusterInstance. LocalCluster cache is a terrible and niche cache that is being deprecated. I reviewed cache uses in areas of GrowthExperiments I'm responsible for, and I do not see any benefits of using a local cluster cache. Let move to the WAN cache, which seems to be a better fit for what we're doing.

@Ladsgroup said in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GrowthExperiments/+/748827/7#message-1bdd570f88402cd35eec06bb64cf04b740ab558c: > The cache for MentorPageMentorManager is ObjectCache::getLocalClusterInstance. LocalCluster cache is a terrible and niche cache that is being deprecated. I reviewed cache uses in areas of GrowthExperiments I'm responsible for, and I do not see any benefits of using a local cluster cache. Let move to the WAN cache, which seems to be a better fit for what we're doing. ==== Classes to fix [x] MentorStore [x] DatabaseMenteeOverviewDataProvider [ ] WikiPageConfigLoader [ ] AqsEditInfoService

@Ladsgroup said in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GrowthExperiments/+/748827/7#message-1bdd570f88402cd35eec06bb64cf04b740ab558c: > The cache for MentorPageMentorManager is ObjectCache::getLocalClusterInstance. LocalCluster cache is a terrible and niche cache that is being deprecated. I reviewed cache uses in areas of GrowthExperiments I'm responsible for, and I do not see any benefits of using a local cluster cache. Let move to the WAN cache, which seems to be a better fit for what we're doing. ==== Classes to fix [x] MentorStore [x] DatabaseMenteeOverviewDataProvider [ ] WikiPageConfigLoader [ ] AqsEditInfoService