opendev/gerrit
Code Issues Proposed changes

OnlineReindexer: log the success/failure numbers on exit

Browse Source 
When the OnlineReindexer fails to reindex all changes it will not
activate the new index schema version. However, it may happen that only
there were only a few failures and that activating the new index schema
makes sense. Help the admins make the decision whether to activate the
new index schema version by logging the success/failure numbers in the
error_log.

Change-Id: I522e236ab1e9b60d5c3c7c215d8308972db45f70
This commit is contained in:
Saša Živkov 2015-03-25 13:50:42 +01:00
parent 61074ca4fb
commit c89b7599ed
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-lucene/src/main/java/com/google/gerrit/lucene/OnlineReindexer.java
View File

@ -79,7 +79,9 @@ public class OnlineReindexer {
ChangeBatchIndexer.Result result = batchIndexer.indexAll(
index, projectCache.all(), -1, -1, null, null);
if (!result.success()) {
log.error("Online reindex of schema version {} failed", version(index));
log.error("Online reindex of schema version {} failed. Successfully"
+ " indexed {} changes, failed to index {} changes",
version(index), result.doneCount(), result.failedCount());
return;
}