opendev/gerrit
Code Issues Proposed changes

Log number of auto-abandoned changes as info

Browse Source 
Change-Id: I61ce1f6450677c3c8b0c03cd17ece6d2fd5cc07f
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
This commit is contained in:
Edwin Kempin
2015-06-10 15:51:10 +02:00
parent 7080ef1f9d
commit 9c717b61a5
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -75,9 +75,11 @@ public class AbandonUtil {
} }
List<ChangeData> changesToAbandon = queryProcessor.enforceVisibility(false) List<ChangeData> changesToAbandon = queryProcessor.enforceVisibility(false)
.queryChanges(queryBuilder.parse(query)).changes(); .queryChanges(queryBuilder.parse(query)).changes();
int count = 0;
for (ChangeData cd : changesToAbandon) { for (ChangeData cd : changesToAbandon) {
try { try {
abandon.abandon(changeControl(cd), cfg.getAbandonMessage(), null); abandon.abandon(changeControl(cd), cfg.getAbandonMessage(), null);
count++;
} catch (ResourceConflictException e) { } catch (ResourceConflictException e) {
// Change was already merged or abandoned. // Change was already merged or abandoned.
} catch (Throwable e) { } catch (Throwable e) {
@@ -86,6 +88,8 @@ public class AbandonUtil {
cd.getId().get()), e); cd.getId().get()), e);
} }
} }
log.info(String.format("Auto-Abandoned %d of %d changes.",
count, changesToAbandon.size()));
} catch (QueryParseException | OrmException e) { } catch (QueryParseException | OrmException e) {
log.error("Failed to query inactive open changes for auto-abandoning.", e); log.error("Failed to query inactive open changes for auto-abandoning.", e);
} }