[HACKERS] Confusing error message in pgbench - Mailing list pgsql-hackers

From Tatsuo Ishii
Subject [HACKERS] Confusing error message in pgbench
Date
Msg-id 20170802.110328.1963639094551443169.t-ishii@sraoss.co.jp
Whole thread Raw
Responses Re: [HACKERS] Confusing error message in pgbench
Re: [HACKERS] Confusing error message in pgbench
List pgsql-hackers
I found an error message in pgbench is quite confusing.

pgbench -S -M extended -c 1 -T 30 test
query mode (-M) should be specified before any transaction scripts (-f or -b)

Since there's no -f or -b option is specified, users will be
confused. Actually the error occurs because pgbench implicitly
introduces a built in script for -S. To eliminate the confusion, I
think the error message should be fixed like this:

query mode (-M) should be specified before transaction type (-S or -N) or any transaction scripts (-f or -b)

Patch attached.

Best regards,
--
Tatsuo Ishii
SRA OSS, Inc. Japan
English: http://www.sraoss.co.jp/index_en.php
Japanese:http://www.sraoss.co.jp
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c
index 4d364a1..f7ef0ed 100644
--- a/src/bin/pgbench/pgbench.c
+++ b/src/bin/pgbench/pgbench.c
@@ -3898,7 +3898,7 @@ main(int argc, char **argv)                benchmarking_option_set = true;                if
(num_scripts> 0)                {
 
-                    fprintf(stderr, "query mode (-M) should be specified before any transaction scripts (-f or
-b)\n");
+                    fprintf(stderr, "query mode (-M) should be specified before transaction type (-S or -N) or any
transactionscripts (-f or -b)\n");                    exit(1);                }                for (querymode = 0;
querymode< NUM_QUERYMODE; querymode++) 

pgsql-hackers by date:

Previous
From: Noah Misch
Date:
Subject: Re: [HACKERS] VACUUM and ANALYZE disagreeing on what reltuples means
Next
From: Amit Kapila
Date:
Subject: Re: [HACKERS] Proposal for CSN based snapshots