pgsql: Use procsignal_sigusr1_handler for auxiliary processes.

From: Robert Haas <rhaas(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Use procsignal_sigusr1_handler for auxiliary processes.
Date: 2019-11-25 22:43:48
Message-ID: E1iZN5A-0006ot-Dq@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Amit Kapila 2019-11-26 04:25:48 pgsql: Don't shut down Gather[Merge] early under Limit.
Previous Message Andrew Dunstan 2019-11-25 21:01:46 pgsql: Close stdin where it's not needed in TestLib.pm procedures