pgsql: Move some code from src/bin/scripts to src/fe_utils to permit re - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: Move some code from src/bin/scripts to src/fe_utils to permit re
Date
Msg-id E1l86F4-0000lk-Eg@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Move some code from src/bin/scripts to src/fe_utils to permit reuse.

The parallel slots infrastructure (which implements client-side
multiplexing of server connections doing similar things, not
threading or multiple processes or anything like that) are moved from
src/bin/scripts/scripts_parallel.c to src/fe_utils/parallel_slot.c.

The functions consumeQueryResult() and processQueryResult() which were
previously part of src/bin/scripts/common.c are now moved into that
file as well, becoming static helper functions. This might need to be
changed in the future, but currently they're not used for anything
else.

Some other functions from src/bin/scripts/common.c are moved to to
src/fe_utils and are split up among several files.  connectDatabase(),
connectMaintenanceDatabase(), and disconnectDatabase() are moved to
connect_utils.c.  executeQuery(), executeCommand(), and
executeMaintenanceCommand() are move to query_utils.c.
handle_help_version_opts() is moved to option_utils.c.

Mark Dilger, reviewed by me. The larger patch series of which this is
a part has also had review from Peter Geoghegan, Andres Freund, Álvaro
Herrera, Michael Paquier, and Amul Sul, but I don't know whether any
of them have reviewed this bit specifically.

Discussion: http://postgr.es/m/12ED3DA8-25F0-4B68-937D-D907CFBF08E7@enterprisedb.com
Discussion: http://postgr.es/m/5F743835-3399-419C-8324-2D424237E999@enterprisedb.com
Discussion: http://postgr.es/m/70655DF3-33CE-4527-9A4D-DDEB582B6BA0@enterprisedb.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/e955bd4b6c2bcdbd253837f6cf4c7520b98e69d4

Modified Files
--------------
src/bin/scripts/Makefile                           |   6 +-
src/bin/scripts/clusterdb.c                        |   2 +
src/bin/scripts/common.c                           | 318 +--------------------
src/bin/scripts/common.h                           |  49 +---
src/bin/scripts/createdb.c                         |   1 +
src/bin/scripts/createuser.c                       |   1 +
src/bin/scripts/dropdb.c                           |   1 +
src/bin/scripts/dropuser.c                         |   1 +
src/bin/scripts/nls.mk                             |   2 +-
src/bin/scripts/pg_isready.c                       |   1 +
src/bin/scripts/reindexdb.c                        |   4 +-
src/bin/scripts/vacuumdb.c                         |   4 +-
src/fe_utils/Makefile                              |   4 +
src/fe_utils/connect_utils.c                       | 181 ++++++++++++
src/fe_utils/option_utils.c                        |  38 +++
.../parallel_slot.c}                               |  63 +++-
src/fe_utils/query_utils.c                         |  92 ++++++
src/include/fe_utils/connect_utils.h               |  48 ++++
src/include/fe_utils/option_utils.h                |  23 ++
.../fe_utils/parallel_slot.h}                      |  13 +-
src/include/fe_utils/query_utils.h                 |  26 ++
src/tools/msvc/Mkvcbuild.pm                        |   5 +-
22 files changed, 498 insertions(+), 385 deletions(-)


pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgsql: Fix backslash-escaping multibyte chars in COPY FROM.
Next
From: Tom Lane
Date:
Subject: pgsql: First-draft release notes for 13.2.