diff options
| author | Fujii Masao <fujii@postgresql.org> | 2021-08-30 21:35:24 +0900 |
|---|---|---|
| committer | Fujii Masao <fujii@postgresql.org> | 2021-08-30 21:35:24 +0900 |
| commit | bfd4567b88496bf9669658e5ab381b296dd9ffe1 (patch) | |
| tree | 9a7a730a487607012e779d5cf01d5f46c5ad18b8 /src | |
| parent | bad6cef32ca2dd60c8be18f95a7157ced189ec99 (diff) | |
| download | postgresql-bfd4567b88496bf9669658e5ab381b296dd9ffe1.tar.gz postgresql-bfd4567b88496bf9669658e5ab381b296dd9ffe1.zip | |
pgbench: Avoid unnecessary measurement of connection delays.
Commit 547f04e734 changed pgbench so that it used the measurement result
of connection delays in its benchmark report only when -C/--connect option
is specified. But previously those delays were unnecessarily measured
even when that option is not specified. Which was a waste of cycles.
This commit improves pgbench so that it avoids such unnecessary measurement.
Back-patch to v14 where commit 547f04e734 first appeared.
Author: Yugo Nagata
Reviewed-by: Fabien COELHO, Asif Rehman, Fujii Masao
Discussion: https://postgr.es/m/20210614151155.a393bc7d8fed183e38c9f52a@sraoss.co.jp
Diffstat (limited to 'src')
| -rw-r--r-- | src/bin/pgbench/pgbench.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c index b0e20c46ae3..bca136bdd58 100644 --- a/src/bin/pgbench/pgbench.c +++ b/src/bin/pgbench/pgbench.c @@ -6603,6 +6603,7 @@ threadRun(void *arg) thread_start = pg_time_now(); thread->started_time = thread_start; + thread->conn_duration = 0; last_report = thread_start; next_report = last_report + (int64) 1000000 * progress; @@ -6626,14 +6627,6 @@ threadRun(void *arg) goto done; } } - - /* compute connection delay */ - thread->conn_duration = pg_time_now() - thread->started_time; - } - else - { - /* no connection delay to record */ - thread->conn_duration = 0; } /* GO */ |