pgsql: initdb: Refactor PG_CMD_PUTS loops - Mailing list pgsql-committers

From Peter Eisentraut
Subject pgsql: initdb: Refactor PG_CMD_PUTS loops
Date
Msg-id E1p2K8G-0029iG-SO@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
initdb: Refactor PG_CMD_PUTS loops

Keeping the SQL commands that initdb runs in string arrays before
feeding them to PG_CMD_PUTS() seems unnecessarily verbose and
inflexible.  In some cases, the array only has one member.  In other
cases, one might want to use PG_CMD_PRINTF() instead, to parametrize a
command, but that would require breaking up the loop or using
workarounds like replace_token().  Unwind all that; it's much simpler
that way.

Reviewed-by: John Naylor <john.naylor@enterprisedb.com>
Reviewed-by: Andrew Dunstan <andrew@dunslane.net>
Discussion: https://www.postgresql.org/message-id/flat/2c50823b-f453-bb97-e38b-34751c51dcdf%40enterprisedb.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1bd47d0dca9f63dc26abc00d4912857440cd101c

Modified Files
--------------
src/bin/initdb/initdb.c | 379 ++++++++++++++++++++++--------------------------
1 file changed, 170 insertions(+), 209 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix Memoize to work with partitionwise joining.
Next
From: Michael Paquier
Date:
Subject: pgsql: Provide test coverage in pg_dump for default behaviors with comp