diff options
| author | Tatsuo Ishii <ishii@postgresql.org> | 2022-04-19 17:04:27 +0900 |
|---|---|---|
| committer | Tatsuo Ishii <ishii@postgresql.org> | 2022-04-19 17:04:27 +0900 |
| commit | a62bff74b135b191de0417d1cc8ec52f144c8fe7 (patch) | |
| tree | 84eb51d9a3cb559791da1e8f61f85f6f53dd8ec4 /src | |
| parent | dd4ab6fd6528e160571986fa8817cee9f2645aa8 (diff) | |
| download | postgresql-a62bff74b135b191de0417d1cc8ec52f144c8fe7.tar.gz postgresql-a62bff74b135b191de0417d1cc8ec52f144c8fe7.zip | |
Fix aggregate logging of pgbench.
Remove meaningless "failures" column from the aggregate logging. It
was just a sum of "serialization failures" and "deadlock failures".
Pointed out by Tom Lane. Patch reviewed by Fabien COELHO.
Discussion: https://postgr.es/m/4183048.1649536705%40sss.pgh.pa.us
Diffstat (limited to 'src')
| -rw-r--r-- | src/bin/pgbench/pgbench.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c index e63cea56a1c..f8bcb1ab6de 100644 --- a/src/bin/pgbench/pgbench.c +++ b/src/bin/pgbench/pgbench.c @@ -4498,7 +4498,6 @@ doLog(TState *thread, CState *st, int64 skipped = 0; int64 serialization_failures = 0; int64 deadlock_failures = 0; - int64 serialization_or_deadlock_failures = 0; int64 retried = 0; int64 retries = 0; @@ -4540,9 +4539,7 @@ doLog(TState *thread, CState *st, serialization_failures = agg->serialization_failures; deadlock_failures = agg->deadlock_failures; } - serialization_or_deadlock_failures = serialization_failures + deadlock_failures; - fprintf(logfile, " " INT64_FORMAT " " INT64_FORMAT " " INT64_FORMAT, - serialization_or_deadlock_failures, + fprintf(logfile, " " INT64_FORMAT " " INT64_FORMAT, serialization_failures, deadlock_failures); |