diff options
author | Dave Borowitz <dborowitz@google.com> | 2012-10-10 15:00:44 -0700 |
---|---|---|
committer | Edwin Kempin <edwin.kempin@sap.com> | 2012-10-11 07:33:03 +0200 |
commit | 8fd5b44d3e388ad9084be033b14660e4d8cd7add (patch) | |
tree | fae6ed518e0372496c8dd3ee9133011b2c973d0f | |
parent | 6fb89cfea4e092f01a27a6f9e39adedcb82f25ca (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.java | 2 |
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 { |