pgsql: Set wal_receiver_create_temp_slot PGC_POSTMASTER

Previous Topic Next Topic
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

pgsql: Set wal_receiver_create_temp_slot PGC_POSTMASTER

Álvaro Herrera
Set wal_receiver_create_temp_slot PGC_POSTMASTER

Commit 329730827848 gave walreceiver the ability to create and use a
temporary replication slot, and made it controllable by a GUC (enabled
by default) that can be changed with SIGHUP.  That's useful but has two
problems: one, it's possible to cause the origin server to fill its disk
if the slot doesn't advance in time; and also there's a disconnect
between state passed down via the startup process and GUCs that
walreceiver reads directly.

We handle the first problem by setting the option to disabled by
default.  If the user enables it, its on their head to make sure that
disk doesn't fill up.

We handle the second problem by passing the flag via startup rather than
having walreceiver acquire it directly, and making it PGC_POSTMASTER
(which ensures a walreceiver always has the fresh value).  A future
commit can relax this (to PGC_SIGHUP again) by having the startup
process signal walreceiver to shutdown whenever the value changes.

Author: Sergei Kornilov <[hidden email]>
Reviewed-by: Michael Paquier <[hidden email]>
Reviewed-by: Álvaro Herrera <[hidden email]>



Modified Files
doc/src/sgml/config.sgml                      |  6 +--
src/backend/access/transam/xlog.c             |  4 +-
src/backend/replication/walreceiver.c         | 62 ++++++++++++---------------
src/backend/replication/walreceiverfuncs.c    | 28 +++++++++---
src/backend/utils/misc/guc.c                  |  4 +-
src/backend/utils/misc/postgresql.conf.sample |  4 +-
src/include/access/xlog.h                     |  1 +
src/include/replication/walreceiver.h         |  4 +-
8 files changed, 63 insertions(+), 50 deletions(-)