opendev/gerrit
Code Issues Proposed changes

Untangle persistent/memory cache implementations from each other

Browse Source 
There is mutual dependency between these 2 implementations
(including Guice module that is responsible for installing them)
and it prevents from providing alternative implementations.

Change-Id: I570fd54adf58c466925f4a31a3bf3cd5ace8742c
Signed-off-by: Jacek Centkowski <jcentkowski@collab.net>
This commit is contained in:
Jacek Centkowski
2018-05-08 18:50:58 +02:00
committed by David Pursehouse
parent 7914c57c59
commit e602b8dacc
7 changed files with 147 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-war/src/main/java/com/google/gerrit/httpd/WebAppInitializer.java
View File

@@ -318,7 +318,8 @@ public class WebAppInitializer extends GuiceServletContextListener implements Fi
modules.add(cfgInjector.getInstance(GerritGlobalModule.class));
modules.add(new SearchingChangeCacheImpl.Module());
modules.add(new InternalAccountDirectory.Module());
modules.add(new DefaultCacheFactory.Module());
modules.add(new DefaultCacheFactory.MemoryCacheModule());
modules.add(new DefaultCacheFactory.PersistentCacheModule());
modules.add(cfgInjector.getInstance(MailReceiver.Module.class));
modules.add(new SmtpEmailSender.Module());
modules.add(new SignedTokenEmailTokenVerifier.Module());