diff options
| author | Tatsuo Ishii <ishii@postgresql.org> | 2013-12-12 19:01:01 +0900 |
|---|---|---|
| committer | Tatsuo Ishii <ishii@postgresql.org> | 2013-12-12 19:07:53 +0900 |
| commit | 27902bc9164d227cbdf1242de218edeaf8cd9f7f (patch) | |
| tree | 9204ce465f24cb2a6699020aabea81b004771f6b | |
| parent | 6dc71a7e773c55cff72fc1f9c960287480ad3130 (diff) | |
| download | postgresql-27902bc9164d227cbdf1242de218edeaf8cd9f7f.tar.gz postgresql-27902bc9164d227cbdf1242de218edeaf8cd9f7f.zip | |
Fix progress logging when scale factor is large.
Integer overflow showed minus percent and minus remaining time something like this.
239300000 of 3800000000 tuples (-48%) done (elapsed 226.86 s, remaining -696.10 s).
| -rw-r--r-- | contrib/pgbench/pgbench.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/contrib/pgbench/pgbench.c b/contrib/pgbench/pgbench.c index 8c202bf8702..5ddea8674ab 100644 --- a/contrib/pgbench/pgbench.c +++ b/contrib/pgbench/pgbench.c @@ -1610,11 +1610,11 @@ init(bool is_no_vacuum) INSTR_TIME_SUBTRACT(diff, start); elapsed_sec = INSTR_TIME_GET_DOUBLE(diff); - remaining_sec = (scale * naccounts - j) * elapsed_sec / j; + remaining_sec = ((double) scale * naccounts - j) * elapsed_sec / j; fprintf(stderr, INT64_FORMAT " of " INT64_FORMAT " tuples (%d%%) done (elapsed %.2f s, remaining %.2f s).\n", j, (int64) naccounts * scale, - (int) (((int64) j * 100) / (naccounts * scale)), + (int) (((int64) j * 100) / (naccounts * (int64) scale)), elapsed_sec, remaining_sec); } /* let's not call the timing for each row, but only each 100 rows */ @@ -1624,14 +1624,14 @@ init(bool is_no_vacuum) INSTR_TIME_SUBTRACT(diff, start); elapsed_sec = INSTR_TIME_GET_DOUBLE(diff); - remaining_sec = (scale * naccounts - j) * elapsed_sec / j; + remaining_sec = ((double) scale * naccounts - j) * elapsed_sec / j; /* have we reached the next interval (or end)? */ if ((j == scale * naccounts) || (elapsed_sec >= log_interval * LOG_STEP_SECONDS)) { fprintf(stderr, INT64_FORMAT " of " INT64_FORMAT " tuples (%d%%) done (elapsed %.2f s, remaining %.2f s).\n", j, (int64) naccounts * scale, - (int) (((int64) j * 100) / (naccounts * scale)), elapsed_sec, remaining_sec); + (int) (((int64) j * 100) / (naccounts * (int64) scale)), elapsed_sec, remaining_sec); /* skip to the next interval */ log_interval = (int) ceil(elapsed_sec / LOG_STEP_SECONDS); |