Re: Fix around conn_duration in pgbench
Fabien COELHO <coelho@cri.ensmp.fr>
From: Fabien COELHO <coelho@cri.ensmp.fr>
To: Fujii Masao <masao.fujii@oss.nttdata.com>
Cc: Yugo NAGATA <nagata@sraoss.co.jp>, Asif Rehman <asifr.rehman@gmail.com>,
PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>
Date: 2021-08-31T05:01:58Z
Lists: pgsql-hackers
>> Ok. That makes sense. The output reports "including connections >> establishing" and "excluding connections establishing" regardless with >> -C, so we should measure delays in the same way. > > On second thought, it's more reasonable and less confusing not to > measure the disconnection delays at all? Since whether the benchmark result > should include the disconnection delays or not is not undocumented, > probably we cannot say strongly the current behavior (i.e., the disconnection > delays are not measured) is a bug. Also since the result has not included > the disconnection delays so far, the proposed change might slightly change > the benchmark numbers reported, which might confuse the users. > ISTM that at least it's unwise to change long-stable branches for this... > Thought? My 0.02€: From a benchmarking perspective, ISTM that it makes sense to include disconnection times, which are clearly linked to connections, especially with -C. So I'd rather have the more meaningful figure even at the price of a small change in an undocumented feature. -- Fabien.
Commits
-
pgbench: Fix bug in measurement of disconnection delays.
- d760d942c73c 14.0 landed
- 4dc528bfa7da 15.0 landed
-
pgbench: Avoid unnecessary measurement of connection delays.
- efe2382d5ade 14.0 landed
- bfd4567b8849 15.0 landed
-
pgbench: Improve time logic.
- 547f04e7348b 14.0 cited