opendev/gerrit
Code Issues Proposed changes

Log changes failing reindex because of missing ref

Browse Source 
Change-Id: Id89d84e3c0e577a244d6941d9929f4b840a4adb9
This commit is contained in:
Martin Fick
2013-06-27 09:55:03 -06:00
committed by Martin Fick
parent 0559859e68
commit 88fecb11ac
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-pgm/src/main/java/com/google/gerrit/pgm/Reindex.java
View File

@@ -335,9 +335,13 @@ public class Reindex extends SiteProgram {
try { try {
Map<String, Ref> refs = repo.getAllRefs(); Map<String, Ref> refs = repo.getAllRefs();
for (Change c : db.changes().byProject(project)) { for (Change c : db.changes().byProject(project)) {
Ref r = refs.get(c.currentPatchSetId().toRefName()); String refName = c.currentPatchSetId().toRefName();
Ref r = refs.get(refName);
if (r != null) { if (r != null) {
byId.put(r.getObjectId(), new ChangeData(c)); byId.put(r.getObjectId(), new ChangeData(c));
} else {
fail("Failed to index change " + c.getId()
+ " (" + refName + " not found)", true, null);
} }
} }
walk(); walk();