pgsql: Use procsignal_sigusr1_handler for auxiliary processes. - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: Use procsignal_sigusr1_handler for auxiliary processes.
Date
Msg-id E1iZN5A-0006ot-Dq@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Use procsignal_sigusr1_handler for auxiliary processes.

AuxiliaryProcessMain does ProcSignalInit, so one might expect that
auxiliary processes would need to respond to SendProcSignal, but none
of the auxiliary processes do that. Change them to use
procsignal_sigusr1_handler instead of their own private handlers so
that they do. Besides seeming more correct, this is also less code. It
shouldn't make any functional difference right now because, as far as
we know, there are no current cases where SendProcSignal targets an
auxiliary process, but there are plans to change that in the future.

Andres Freund

Discussion: http://postgr.es/m/20181030051643.elbxjww5jjgnjaxg@alap3.anarazel.de

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/0d3c3aae3366891f1c3d6bac326070660be36f76

Modified Files
--------------
src/backend/postmaster/bgwriter.c     | 20 +++-----------------
src/backend/postmaster/checkpointer.c | 15 ++-------------
src/backend/postmaster/startup.c      | 15 ++-------------
src/backend/postmaster/walwriter.c    | 15 ++-------------
src/backend/replication/walreceiver.c | 15 ++-------------
5 files changed, 11 insertions(+), 69 deletions(-)


pgsql-committers by date:

Previous
From: Andrew Dunstan
Date:
Subject: pgsql: Close stdin where it's not needed in TestLib.pm procedures
Next
From: Amit Kapila
Date:
Subject: pgsql: Don't shut down Gather[Merge] early under Limit.