[COMMITTERS] pgsql: Use standard interrupt handling in logical replicationlauncher. - Mailing list pgsql-committers

From Andres Freund
Subject [COMMITTERS] pgsql: Use standard interrupt handling in logical replicationlauncher.
Date
Msg-id E1dJ6Bw-0006wW-5b@gemulon.postgresql.org
Whole thread Raw
Responses Re: [COMMITTERS] pgsql: Use standard interrupt handling in logical replication launcher.  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
Use standard interrupt handling in logical replication launcher.

Previously the exit handling was only able to exit from within the
main loop, and not from within the backend code it calls.  Fix that by
using the standard die() SIGTERM handler, and adding the necessary
CHECK_FOR_INTERRUPTS() call.

This requires adding yet another process-type-specific branch to
ProcessInterrupts(), which hints that we probably should generalize
that handling.  But that's work for another day.

Author: Petr Jelinek
Reviewed-By: Andres Freund
Discussion: https://postgr.es/m/fe072153-babd-3b5d-8052-73527a6eb657@2ndquadrant.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/2c48f5db64b1b999b08052115a5ce873343c372a

Modified Files
--------------
src/backend/replication/logical/launcher.c | 41 ++++++++++++------------------
src/backend/tcop/postgres.c                |  9 +++++++
src/include/replication/logicallauncher.h  |  2 ++
3 files changed, 27 insertions(+), 25 deletions(-)


pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: [COMMITTERS] pgsql: Again report a useful error message when walreceiver'sconnectio
Next
From: Heikki Linnakangas
Date:
Subject: [COMMITTERS] pgsql: Fix script name in README.