opendev/gerrit
Code Issues Proposed changes

Extend MigrateAccountPatchReviewDb summary with stats

Browse Source 
Add the following information:
1. how many rows were migrated
2. how much time it took
3. what was migration speed (rows per/sec)

So that it is more consistent with Reindex operation output.

Change-Id: If49da15ed9806f2b0dd2e63911445a55785a5cfb
This commit is contained in:
Jacek Centkowski 2017-12-05 12:34:46 +01:00
parent 69678200e8
commit b04decf8b5
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -15,6 +15,7 @@
package com.google.gerrit.pgm;
import com.google.auto.value.AutoValue;
import com.google.common.base.Stopwatch;
import com.google.common.base.Strings;
import com.google.gerrit.pgm.util.SiteProgram;
import com.google.gerrit.server.config.GerritServerConfig;
@ -31,6 +32,7 @@ import java.sql.SQLException;
import java.sql.Statement;
import java.util.ArrayList;
import java.util.List;
import java.util.concurrent.TimeUnit;
import org.eclipse.jgit.lib.Config;
import org.kohsuke.args4j.Option;
@ -91,6 +93,7 @@ public class MigrateAccountPatchReviewDb extends SiteProgram {
+ "(?, ?, ?, ?)")) {
targetCon.setAutoCommit(false);
long offset = 0;
Stopwatch sw = Stopwatch.createStarted();
List<Row> rows = selectRows(sourceStmt, offset);
while (!rows.isEmpty()) {
insertRows(targetCon, targetStmt, rows);
@ -98,7 +101,8 @@ public class MigrateAccountPatchReviewDb extends SiteProgram {
System.out.printf("%8d rows migrated\n", offset);
rows = selectRows(sourceStmt, offset);
}
System.out.println("Done");
double t = sw.elapsed(TimeUnit.MILLISECONDS) / 1000d;
System.out.printf("Migrated %d rows in %.01fs (%.01f/s)\n", offset, t, offset / t);
}
return 0;
}