summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDave Borowitz <dborowitz@google.com>2012-10-10 15:00:44 -0700
committerEdwin Kempin <edwin.kempin@sap.com>2012-10-11 07:33:03 +0200
commit8fd5b44d3e388ad9084be033b14660e4d8cd7add (patch)
treefae6ed518e0372496c8dd3ee9133011b2c973d0f
parent6fb89cfea4e092f01a27a6f9e39adedcb82f25ca (diff)
Call inserter.flush() when creating code review notes
Failing to do this can cause the notes blobs to not get written to storage, particularly with JGit's DfsInserter. Change-Id: I28168374ddf9f89f5107a91bfba44f345ef395cf
-rw-r--r--gerrit-server/src/main/java/com/google/gerrit/server/git/CreateCodeReviewNotes.java2
1 files changed, 2 insertions, 0 deletions
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/git/CreateCodeReviewNotes.java b/gerrit-server/src/main/java/com/google/gerrit/server/git/CreateCodeReviewNotes.java
index b067a4915d..9a0fe17e0c 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/git/CreateCodeReviewNotes.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/git/CreateCodeReviewNotes.java
@@ -117,6 +117,7 @@ public class CreateCodeReviewNotes {
NotesBranchUtil notesBranchUtil = notesBranchUtilFactory.create(db);
notesBranchUtil.commitAllNotes(notes, REFS_NOTES_REVIEW, author,
message.toString());
+ inserter.flush();
} catch (IOException e) {
throw new CodeReviewNoteCreationException(e);
} catch (ConcurrentRefUpdateException e) {
@@ -151,6 +152,7 @@ public class CreateCodeReviewNotes {
NotesBranchUtil notesBranchUtil = notesBranchUtilFactory.create(db);
notesBranchUtil.commitAllNotes(notes, REFS_NOTES_REVIEW, author,
commitMessage);
+ inserter.flush();
} catch (ConcurrentRefUpdateException e) {
throw new CodeReviewNoteCreationException(e);
} finally {