opendev/gerrit
Code Issues Proposed changes

Merge "AllChangesIndexer: Don't abort when failing to open repository" into stable-2.14

Browse Source
This commit is contained in:
David Pursehouse
2018-05-16 11:17:07 +00:00
committed by Gerrit Code Review
parent 104ea59499 83eaad4232
commit 4731b1c664
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-server/src/main/java/com/google/gerrit/server/index/change/AllChangesIndexer.java
View File

@@ -118,15 +118,13 @@ public class AllChangesIndexer extends SiteIndexer<Change.Id, ChangeData, Change
pm.beginTask("Collecting projects", ProgressMonitor.UNKNOWN); pm.beginTask("Collecting projects", ProgressMonitor.UNKNOWN);
SortedSet<ProjectHolder> projects = new TreeSet<>(); SortedSet<ProjectHolder> projects = new TreeSet<>();
int changeCount = 0; int changeCount = 0;
Stopwatch sw = Stopwatch.createStarted();
for (Project.NameKey name : projectCache.all()) { for (Project.NameKey name : projectCache.all()) {
try (Repository repo = repoManager.openRepository(name)) { try (Repository repo = repoManager.openRepository(name)) {
int size = ChangeNotes.Factory.scan(repo).size(); int size = ChangeNotes.Factory.scan(repo).size();
changeCount += size; changeCount += size;
projects.add(new ProjectHolder(name, size)); projects.add(new ProjectHolder(name, size));
} catch (IOException e) { } catch (IOException e) {
log.error("Error collecting projects", e); log.error("Error collecting changes for project {}", name, e);
return new Result(sw, false, 0, 0);
} }
pm.update(1); pm.update(1);
} }