RE: pgbench: option delaying queries till connections establishment?
Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com>
From: "kuroda.hayato@fujitsu.com" <kuroda.hayato@fujitsu.com>
To: 'Fabien COELHO' <coelho@cri.ensmp.fr>
Cc: Andres Freund <andres@anarazel.de>, Daniel Gustafsson <daniel@yesql.se>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Date: 2020-11-06T02:15:10Z
Lists: pgsql-hackers
Dear Fabien, > Indeed. I scanned the file but did not find other places that needed > updating. > Yes. > Not sure either. I'm not for having too many braces anyway, so I removed > them. I checked your fixes and I think it's OK. Finally, please move the doc fixes to patch B in order to separate patches completely. > Indeed. I took your next patch with an added explanation. I'm unclear > whether proceeding makes much sense though, that is some thread would run > the test and other would have aborted. Hmmm. Your comment looks good, thanks. In the previous version pgbench starts benchmarking even if some connections fail. And users can notice the connection failure by stderr output. Hence the current specification may be enough. If you agree, please remove the following lines: ``` + * It is unclear whether it is worth doing anything rather than + * coldly exiting with an error message. ``` > ISTM that there is another patch in the queue which needs barriers to > delay some initialization so as to fix a corner case bug, in which case > the behavior would be mandatory. The current submission could add an > option to skip the barrier synchronization, but I'm not enthousiastic to > add an option and remove it shortly later. Could you tell me which patch you mention? Basically I agree what you say, but I want to check it. Hayato Kuroda FUJITSU LIMITED
Commits
-
Fix new pthread code to respect --disable-thread-safety.
- de91c3b976cf 14.0 landed
-
Add missing pthread_barrier_t.
- 44bf3d5083e1 14.0 landed
-
pgbench: Improve time logic.
- 547f04e7348b 14.0 landed
-
pgbench: Refactor thread portability support.
- b1d6a8f86813 14.0 landed
-
pgbench: Synchronize client threads.
- aeb57af8e640 14.0 landed
-
Fix bogus logic for skipping unnecessary partcollation dependencies.
- 84d514887f9c 12.0 cited