opendev/gerrit
Code Issues Proposed changes

ChangeInserter: Remove stray beginTransaction call

Browse Source 
Transactions are now handled by BatchUpdate; this one was accidentally
left in.

Change-Id: I283d4d6b87f3eaffa82f54c0e551d4ab4a7529df
This commit is contained in:
Dave Borowitz
2015-10-12 15:34:15 -04:00
parent b729d3e82b
commit 02481e7ac7
1 changed files with 0 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeInserter.java
View File

@@ -269,7 +269,6 @@ public class ChangeInserter extends BatchUpdate.InsertChangeOp {
ReviewDb db = ctx.getDb(); ReviewDb db = ctx.getDb();
ChangeControl ctl = ctx.getChangeControl(); ChangeControl ctl = ctx.getChangeControl();
ChangeUpdate update = ctx.getChangeUpdate(); ChangeUpdate update = ctx.getChangeUpdate();
db.changes().beginTransaction(change.getId());
ChangeUtil.insertAncestors(db, patchSet.getId(), commit); ChangeUtil.insertAncestors(db, patchSet.getId(), commit);
if (patchSet.getGroups() == null) { if (patchSet.getGroups() == null) {
patchSet.setGroups(GroupCollector.getDefaultGroups(patchSet)); patchSet.setGroups(GroupCollector.getDefaultGroups(patchSet));