commit 751a0c40e28a84157d01219774b69abd873afa5f
author David Pursehouse <david.pursehouse@sonymobile.com> Mon Jul 13 11:10:22 2015 +0900
committer David Pursehouse <david.pursehouse@sonymobile.com> Mon Jul 13 12:31:42 2015 +0900
tree 5f54e1cd5c7cc4d27ecc8e783fc11d0c2aa6b3d5
parent 47f15bed74d1a26573d90ce5251fc646db9f3036

Bind OnlineReindexer.Factory in LuceneIndexModule

The OnlineReindexer.Factory is only bound in MultiVersionModule.

This causes some of the acceptance tests to fail with the error
"no implementation bound" because the SingleVersionModule is being
used, which does not bind OnlineReindexer.Factory.

Bind it in LuceneIndexModule instead, so it's used in both cases.

Change-Id: I6df22bb405621485e5dcba2a5d0e0a14de372f8a

gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneIndexModule.java

diff --git a/gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneIndexModule.java b/gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneIndexModule.java
index 672a7c9..3b56fc0 100644
--- a/gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneIndexModule.java
+++ b/gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneIndexModule.java
@@ -46,6 +46,7 @@
   protected void configure() {
     bind(IndexConfig.class).toInstance(IndexConfig.createDefault());
     factory(LuceneChangeIndex.Factory.class);
+    factory(OnlineReindexer.Factory.class);
     install(new IndexModule(threads));
     if (singleVersion == null && base == null) {
       install(new MultiVersionModule());
@@ -57,7 +58,6 @@
   private class MultiVersionModule extends LifecycleModule {
     @Override
     public void configure() {
-      factory(OnlineReindexer.Factory.class);
       listener().to(LuceneVersionManager.class);
     }
   }