opendev/gerrit
Code Issues Proposed changes

Merge "Migrate more callers to ChangeNotes.Factory#createChecked"

Browse Source
This commit is contained in:
David Pursehouse
2016-02-24 00:30:04 +00:00
committed by Gerrit Code Review
parent 30ecc2823c 9219e33eab
commit 4efa2efa07
6 changed files with 54 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-server/src/main/java/com/google/gerrit/server/index/ReindexAfterUpdate.java
View File

@@ -27,6 +27,7 @@ import com.google.gerrit.reviewdb.client.RefNames;
import com.google.gerrit.reviewdb.server.ReviewDb;
import com.google.gerrit.server.git.QueueProvider.QueueType;
import com.google.gerrit.server.notedb.ChangeNotes;
import com.google.gerrit.server.project.NoSuchChangeException;
import com.google.gerrit.server.query.change.InternalChangeQuery;
import com.google.gerrit.server.util.ManualRequestContext;
import com.google.gerrit.server.util.OneOffRequestContext;
@@ -146,11 +147,12 @@ public class ReindexAfterUpdate implements GitReferenceUpdatedListener {
}
@Override
protected Void impl(RequestContext ctx) throws OrmException, IOException {
protected Void impl(RequestContext ctx)
throws OrmException, IOException, NoSuchChangeException {
// Reload change, as some time may have passed since GetChanges.
ReviewDb db = ctx.getReviewDbProvider().get();
Change c = notesFactory
.create(db, new Project.NameKey(event.getProjectName()), id)
.createChecked(db, new Project.NameKey(event.getProjectName()), id)
.getChange();
indexerFactory.create(executor, indexes).index(db, c);
return null;