commit bfec8349dcf37e5a6f3fa08adab847682ea2a11f
author Dave Borowitz <dborowitz@google.com> Wed Apr 20 18:40:08 2016 -0400
committer Dave Borowitz <dborowitz@google.com> Fri Apr 22 10:56:17 2016 -0400
tree 9c0218986e3e58c33a209a0f3a6cb23753675718
parent f819a4dc7aea741f70c12cd21e332e7b07e752df

ChangeRebuilderImpl: Actually save data

I missed calling execute() this when refactoring to add a method that
takes a NoteDbUpdateManager. This was causing RebuildNoteDb to finish
unrealistically quickly.

Change-Id: I21eb436259ae1d74f16351ce064345e123963943

gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeRebuilderImpl.java

diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeRebuilderImpl.java b/gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeRebuilderImpl.java
index ee00093..4a0d56b 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeRebuilderImpl.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeRebuilderImpl.java
@@ -156,7 +156,9 @@
     NoteDbUpdateManager manager =
         updateManagerFactory.create(change.getProject());
     buildUpdates(manager, ChangeBundle.fromReviewDb(db, changeId));
-    return execute(db, changeId, manager);
+    NoteDbChangeState result = execute(db, changeId, manager);
+    manager.execute();
+    return result;
   }
 
   private NoteDbChangeState execute(ReviewDb db, Change.Id changeId,