From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Perform an immediate shutdown if the postmaster.pid file is remo |
Date: | 2015-10-06 21:16:07 |
Message-ID: | E1ZjZah-0007cC-Ro@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Perform an immediate shutdown if the postmaster.pid file is removed.
The postmaster now checks every minute or so (worst case, at most two
minutes) that postmaster.pid is still there and still contains its own PID.
If not, it performs an immediate shutdown, as though it had received
SIGQUIT.
The original goal behind this change was to ensure that failed buildfarm
runs would get fully cleaned up, even if the test scripts had left a
postmaster running, which is not an infrequent occurrence. When the
buildfarm script removes a test postmaster's $PGDATA directory, its next
check on postmaster.pid will fail and cause it to exit. Previously, manual
intervention was often needed to get rid of such orphaned postmasters,
since they'd block new test postmasters from obtaining the expected socket
address.
However, by checking postmaster.pid and not something else, we can provide
additional robustness: manual removal of postmaster.pid is a frequent DBA
mistake, and now we can at least limit the damage that will ensue if a new
postmaster is started while the old one is still alive.
Back-patch to all supported branches, since we won't get the desired
improvement in buildfarm reliability otherwise.
Branch
------
REL9_3_STABLE
Details
-------
http://git.postgresql.org/pg/commitdiff/31bc563b9be306623c5e9a52816b432945fa6df9
Modified Files
--------------
src/backend/postmaster/postmaster.c | 52 ++++++++++++++++++++------
src/backend/utils/init/miscinit.c | 70 +++++++++++++++++++++++++++++++++++
src/include/miscadmin.h | 1 +
3 files changed, 112 insertions(+), 11 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | pgsql | 2015-10-06 21:19:08 | pgsql: Tag refs/tags/REL9_5_BETA1 was created |
Previous Message | Robert Haas | 2015-10-06 19:52:05 | pgsql: Remove more volatile qualifiers. |