opendev/gerrit
Code Issues Proposed changes

Install DiffExecutorModule in batch programs for reindex

Browse Source 
This change fixes the broken RebuildNotedbIT test, it was missing
the DiffExecutor binding.

Change-Id: I293935800f43035089e599e8d16903c054863e4d
This commit is contained in:
Shawn Pearce
2015-03-23 11:39:41 -07:00
parent 4c4452abe5
commit 953ca85838
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-pgm/src/main/java/com/google/gerrit/pgm/util/BatchProgramModule.java
View File

@@ -47,6 +47,7 @@ import com.google.gerrit.server.git.TagCache;
import com.google.gerrit.server.group.GroupModule;
import com.google.gerrit.server.mail.ReplacePatchSetSender;
import com.google.gerrit.server.notedb.NoteDbModule;
import com.google.gerrit.server.patch.DiffExecutorModule;
import com.google.gerrit.server.patch.PatchListCacheImpl;
import com.google.gerrit.server.project.ChangeControl;
import com.google.gerrit.server.project.CommentLinkInfo;
@@ -84,7 +85,9 @@ public class BatchProgramModule extends FactoryModule {
@Override
protected void configure() {
install(reviewDbModule);
install(new DiffExecutorModule());
install(PatchListCacheImpl.module());
// Plugins are not loaded and we're just running through each change
// once, so don't worry about cache removal.
bind(new TypeLiteral<DynamicSet<CacheRemovalListener>>() {})