opendev/gerrit
Code Issues Proposed changes

Remove unneeded nested MultiVersionModule class

Browse Source 
This also align LuceneIndexModule with ElasticIndexModule which will
make it easier to refactor them to remove duplication.

Change-Id: Iaf0abdf0cfe5d9b0374d6f271da572bcd8832e4b
This commit is contained in:
Hugo Arès 2018-04-10 20:20:08 -04:00
parent 37d47e9c6e
commit 532ea15eac
1 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneIndexModule.java
View File

@ -74,7 +74,8 @@ public class LuceneIndexModule extends LifecycleModule {
install(new IndexModule(threads)); install(new IndexModule(threads));
if (singleVersions == null) { if (singleVersions == null) {
install(new MultiVersionModule()); bind(AbstractVersionManager.class).to(LuceneVersionManager.class);
listener().to(LuceneVersionManager.class);
} else { } else {
install(new SingleVersionModule(singleVersions)); install(new SingleVersionModule(singleVersions));
} }
@ -87,12 +88,4 @@ public class LuceneIndexModule extends LifecycleModule {
cfg.getInt("index", "maxTerms", BooleanQuery.getMaxClauseCount())); cfg.getInt("index", "maxTerms", BooleanQuery.getMaxClauseCount()));
return IndexConfig.fromConfig(cfg); return IndexConfig.fromConfig(cfg);
} }
private static class MultiVersionModule extends LifecycleModule {
@Override
public void configure() {
bind(AbstractVersionManager.class).to(LuceneVersionManager.class);
listener().to(LuceneVersionManager.class);
}
}
} }