Re: signal and startup process

Lists: pgsql-hackers
From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: signal and startup process
Date: 2010-10-14 03:57:52
Message-ID: AANLkTi=aHWc0=rOgtJvwkc5=1y5e2MciXNa=D=LOW6wr@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Hi,

It takes nonsensically extra several seconds to shut down the standby server.
This is because the startup process cannot respond immediately the SIGTERM
signal sent by postmaster for the shutdown while it's sleeping on WaitLatch.
This delays the shutdown for up to the sleep time on WaitLatch (currently
5 seconds).

The signal handlers of the startup process should call SetLatch to respond
the signals even while it's sleeping on WaitLatch, like those of walsender
already do. The attached patch does this.

Regards,

--
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

Attachment Content-Type Size
signal_handler_wakeup_recovery_v1.patch application/octet-stream 528 bytes

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: signal and startup process
Date: 2010-10-14 18:17:58
Message-ID: 1287080278.1725.1092.camel@ebony
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Committed.

On Thu, 2010-10-14 at 12:57 +0900, Fujii Masao wrote:
> Hi,
>
> It takes nonsensically extra several seconds to shut down the standby server.
> This is because the startup process cannot respond immediately the SIGTERM
> signal sent by postmaster for the shutdown while it's sleeping on WaitLatch.
> This delays the shutdown for up to the sleep time on WaitLatch (currently
> 5 seconds).
>
> The signal handlers of the startup process should call SetLatch to respond
> the signals even while it's sleeping on WaitLatch, like those of walsender
> already do. The attached patch does this.
>
> Regards,
>

--
Simon Riggs www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Training and Services