pgsql: Avoid unexpected shutdown when sync_replication_slots is enabled - Mailing list pgsql-committers

From Fujii Masao
Subject pgsql: Avoid unexpected shutdown when sync_replication_slots is enabled
Date
Msg-id E1uitlH-000hCA-0Y@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Avoid unexpected shutdown when sync_replication_slots is enabled.

Previously, enabling sync_replication_slots while wal_level was not set
to logical could cause the server to shut down. This was because
the postmaster performed a configuration check before launching
the slot synchronization worker and raised an ERROR if the settings
were incompatible. Since ERROR is treated as FATAL in the postmaster,
this resulted in the entire server shutting down unexpectedly.

This commit changes the postmaster to log that message with a LOG-level
instead of raising an ERROR, allowing the server to continue running
even with the misconfiguration.

Back-patch to v17, where slot synchronization was introduced.

Reported-by: Hugo DUBOIS <hdubois@scaleway.com>
Author: Fujii Masao <masao.fujii@gmail.com>
Reviewed-by: Hugo DUBOIS <hdubois@scaleway.com>
Reviewed-by: Shveta Malik <shveta.malik@gmail.com>
Discussion: https://postgr.es/m/CAH0PTU_pc3oHi__XESF9ZigCyzai1Mo3LsOdFyQA4aUDkm01RA@mail.gmail.com
Backpatch-through: 17

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/4614d53d4ef4d2249df45adedd85da8129feee94

Modified Files
--------------
src/backend/replication/logical/slotsync.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)


pgsql-committers by date:

Previous
From: Álvaro Herrera
Date:
Subject: pgsql: doc: mention unusability of dropped CHECK to verify NOT NULL
Next
From: Fujii Masao
Date:
Subject: pgsql: Avoid unexpected shutdown when sync_replication_slots is enabled