From ee6bbd43e5f4ebdc5d01dd9b7aeac15abb7dfd2b Mon Sep 17 00:00:00 2001
From: Shawn Pearce <sop@google.com>
Date: Fri, 6 Dec 2013 11:30:15 -0800
Subject: [PATCH] Initial change messages should go with the insert

Join these together into one batch write to the database
when running on gerrit-review.

Change-Id: I25e473854c6bdd707fd105dd71fe3533c34c26c4
---
 .../com/google/gerrit/server/change/ChangeInserter.java     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeInserter.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeInserter.java
index 95a7a58bf9..afddd1d46d 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeInserter.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeInserter.java
@@ -162,13 +162,13 @@ public class ChangeInserter {
       LabelTypes labelTypes = refControl.getProjectControl().getLabelTypes();
       approvalsUtil.addReviewers(labelTypes, change, patchSet, patchSetInfo,
           reviewers, Collections.<Account.Id> emptySet());
+      if (changeMessage != null) {
+        db.changeMessages().insert(Collections.singleton(changeMessage));
+      }
       db.commit();
     } finally {
       db.rollback();
     }
-    if (changeMessage != null) {
-      db.changeMessages().insert(Collections.singleton(changeMessage));
-    }
 
     CheckedFuture<?, IOException> f =
         mergeabilityChecker.updateAndIndexAsync(change);