Allow empty queries in pgbench.

tglsfdc · tglsfdc · commit 6471045230f5 · 2016-08-19T17:32:59.000-04:00
This might have been too much of a foot-gun before 9.6, but with the
new commands-end-at-semicolons parsing rule, the only way to get an
empty query into a script is to explicitly write an extra ";".
So we may as well allow the case.

Fabien Coelho

Patch: &lt;alpine.DEB.2.20.1607090922170.3412@sto&gt;
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c
@@ -1898,6 +1898,7 @@ doCustom(TState *thread, CState *st, StatsData *agg)
 			{
 				case PGRES_COMMAND_OK:
 				case PGRES_TUPLES_OK:
+				case PGRES_EMPTY_QUERY:
 					break;		/* OK */
 				default:
 					fprintf(stderr, "client %d aborted in state %d: %s",

Original file line number	Diff line number	Diff line change
`@@ -1898,6 +1898,7 @@ doCustom(TState thread, CState st, StatsData *agg)`
`1898`	`1898`	`{`
`1899`	`1899`	`case PGRES_COMMAND_OK:`
`1900`	`1900`	`case PGRES_TUPLES_OK:`
	`1901`	`+ case PGRES_EMPTY_QUERY:`
`1901`	`1902`	`break; /* OK */`
`1902`	`1903`	`default:`
`1903`	`1904`	`fprintf(stderr, "client %d aborted in state %d: %s",`