opendev/gerrit
Code Issues Proposed changes

Merge "Handle RepositoryNotFoundException when running reindex" into stable-2.8

Browse Source
This commit is contained in:
David Pursehouse
2013-11-27 06:35:24 +00:00
committed by Gerrit Code Review
parent 0c7f247dba 3a20ba9822
commit 07a7a52f88
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
gerrit-server/src/main/java/com/google/gerrit/server/index/ChangeBatchIndexer.java
View File

@@ -40,6 +40,7 @@ import com.google.inject.Provider;
import org.eclipse.jgit.diff.DiffEntry; import org.eclipse.jgit.diff.DiffEntry;
import org.eclipse.jgit.diff.DiffFormatter; import org.eclipse.jgit.diff.DiffFormatter;
import org.eclipse.jgit.errors.RepositoryNotFoundException;
import org.eclipse.jgit.lib.Constants; import org.eclipse.jgit.lib.Constants;
import org.eclipse.jgit.lib.NullProgressMonitor; import org.eclipse.jgit.lib.NullProgressMonitor;
import org.eclipse.jgit.lib.ObjectId; import org.eclipse.jgit.lib.ObjectId;
@@ -203,8 +204,9 @@ public class ChangeBatchIndexer {
@Override @Override
public Void call() throws Exception { public Void call() throws Exception {
Multimap<ObjectId, ChangeData> byId = ArrayListMultimap.create(); Multimap<ObjectId, ChangeData> byId = ArrayListMultimap.create();
Repository repo = repoManager.openRepository(project); Repository repo = null;
try { try {
repo = repoManager.openRepository(project);
Map<String, Ref> refs = repo.getRefDatabase().getRefs(ALL); Map<String, Ref> refs = repo.getRefDatabase().getRefs(ALL);
for (Change c : db.get().changes().byProject(project)) { for (Change c : db.get().changes().byProject(project)) {
Ref r = refs.get(c.currentPatchSetId().toRefName()); Ref r = refs.get(c.currentPatchSetId().toRefName());
@@ -214,8 +216,12 @@ public class ChangeBatchIndexer {
} }
new ProjectIndexer(indexer, byId, repo, done, failed, verboseWriter) new ProjectIndexer(indexer, byId, repo, done, failed, verboseWriter)
.call(); .call();
} catch(RepositoryNotFoundException rnfe) {
log.error(rnfe.getMessage());
} finally { } finally {
if (repo != null) {
repo.close(); repo.close();
}
// TODO(dborowitz): Opening all repositories in a live server may be // TODO(dborowitz): Opening all repositories in a live server may be
// wasteful; see if we can determine which ones it is safe to close // wasteful; see if we can determine which ones it is safe to close
// with RepositoryCache.close(repo). // with RepositoryCache.close(repo).