Redesign archive modules
authorMichael Paquier <michael@paquier.xyz>
Fri, 17 Feb 2023 05:26:42 +0000 (14:26 +0900)
committerMichael Paquier <michael@paquier.xyz>
Fri, 17 Feb 2023 05:26:42 +0000 (14:26 +0900)
commit35739b87dcfef9fc0186aca659f262746fecd778
treed0b8e0c9698f1e0096a6ba90808a405246923fe7
parentd2ea2d310dfdc40328aca5b6c52225de78432e01
Redesign archive modules

A new callback named startup_cb, called shortly after a module is
loaded, is added.  This makes possible the initialization of any
additional state data required by a module.  This initial state data can
be saved in a ArchiveModuleState, that is now passed down to all the
callbacks that can be defined in a module.  With this design, it is
possible to have a per-module state, aimed at opening the door to the
support of more than one archive module.

The initialization of the callbacks is changed so as
_PG_archive_module_init() does not anymore give in input a
ArchiveModuleCallbacks that a module has to fill in with callback
definitions.  Instead, a module now needs to return a const
ArchiveModuleCallbacks.

All the structure and callback definitions of archive modules are moved
into their own header, named archive_module.h, from pgarch.h.
Command-based archiving follows the same line, with a new set of files
named shell_archive.{c,h}.

There are a few more items that are under discussion to improve the
design of archive modules, like the fact that basic_archive calls
sigsetjmp() by itself to define its own error handling flow.  These will
be adjusted later, the changes done here cover already a good portion
of what has been discussed.

Any modules created for v15 will need to be adjusted to this new
design.

Author: Nathan Bossart
Reviewed-by: Andres Freund
Discussion: https://postgr.es/m/20230130194810.6fztfgbn32e7qarj@awork3.anarazel.de
15 files changed:
contrib/basic_archive/basic_archive.c
doc/src/sgml/archive-modules.sgml
src/backend/Makefile
src/backend/archive/Makefile [new file with mode: 0644]
src/backend/archive/meson.build [new file with mode: 0644]
src/backend/archive/shell_archive.c [moved from src/backend/postmaster/shell_archive.c with 73% similarity]
src/backend/meson.build
src/backend/postmaster/Makefile
src/backend/postmaster/meson.build
src/backend/postmaster/pgarch.c
src/backend/utils/misc/guc_tables.c
src/include/archive/archive_module.h [new file with mode: 0644]
src/include/archive/shell_archive.h [new file with mode: 0644]
src/include/postmaster/pgarch.h
src/tools/pgindent/typedefs.list