diff options
| author | Tatsuo Ishii <ishii@postgresql.org> | 2015-09-30 10:36:23 +0900 |
|---|---|---|
| committer | Tatsuo Ishii <ishii@postgresql.org> | 2015-09-30 10:36:23 +0900 |
| commit | 3c4c5acc40720a178e06da6258ce20d3c6e025ab (patch) | |
| tree | 85faa950e26b53c182105dd5f103e8d958ca215c | |
| parent | d8c7bb21ea2a3122cac96b2996d3122f25c160c3 (diff) | |
| download | postgresql-3c4c5acc40720a178e06da6258ce20d3c6e025ab.tar.gz postgresql-3c4c5acc40720a178e06da6258ce20d3c6e025ab.zip | |
Fix incorrect tps number calculation in "excluding connections establishing".
The tolerance (larger than actual tps number) increases as the number
of threads decreases. The bug has been there since the thread support
was introduced in 9.0. Because back patching introduces incompatible
behavior changes regarding the tps number, the fix is committed to
master and 9.5 stable branches only.
Problem spotted by me and fix proposed by Fabien COELHO. Note that his
original patch included more than fixes (a code re-factoring) which is
not related to the problem and I omitted the part.
| -rw-r--r-- | src/bin/pgbench/pgbench.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c index 89559e78da1..f7125d25b0e 100644 --- a/src/bin/pgbench/pgbench.c +++ b/src/bin/pgbench/pgbench.c @@ -2631,7 +2631,7 @@ printResults(int ttype, int64 normal_xacts, int nclients, time_include = INSTR_TIME_GET_DOUBLE(total_time); tps_include = normal_xacts / time_include; tps_exclude = normal_xacts / (time_include - - (INSTR_TIME_GET_DOUBLE(conn_total_time) / nthreads)); + (INSTR_TIME_GET_DOUBLE(conn_total_time) / nclients)); if (ttype == 0) s = "TPC-B (sort of)"; |