]> granicus.if.org Git - postgresql/commit
Remove pgbench's restriction on placement of -M switch.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 11 Aug 2017 19:19:40 +0000 (15:19 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 11 Aug 2017 19:19:40 +0000 (15:19 -0400)
commit79681844297a9d499a10094a17e52b365afdd8bb
tree0a0f1d6bb451fc9020c6c07154f191f292b9b836
parenta1ef920e27ba6ab3602aaf6d6751d8628fac1af8
Remove pgbench's restriction on placement of -M switch.

Previously the -M switch had to appear before any switch that directly
or indirectly specified a benchmarking script.  This was both confusing
and inadequately documented, as per gripe from Tatsuo Ishii.  We can
remove the restriction at the cost of making an extra pass over the
lists of SQL commands, which seems like a cheap price (the string scans
themselves likely cost much more).  The change is just to not extract
parameters from the SQL commands until we have finished parsing the
switches and know the final value of -M.

Per discussion, we'll treat this as a low-grade bug fix and sneak it
into v10, rather than holding it for v11.

Tom Lane, reviewed by Tatsuo Ishii and Fabien Coelho

Discussion: https://postgr.es/m/20170802.110328.1963639094551443169.t-ishii@sraoss.co.jp
Discussion: https://postgr.es/m/10208.1502465077@sss.pgh.pa.us
src/bin/pgbench/pgbench.c