]> granicus.if.org Git - postgresql/commit
Avoid calling kill() in a postmaster signal handler.
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Mon, 24 Aug 2009 17:23:02 +0000 (17:23 +0000)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Mon, 24 Aug 2009 17:23:02 +0000 (17:23 +0000)
commit45f9b4646f2f5608d9dbf6357f5a317edcb1d7da
tree37a79649f33ce2d93a4f28bb2620eb35c4db5088
parent9b708f1f57e8303677f40e7773a3aee716f18a8d
Avoid calling kill() in a postmaster signal handler.

This causes problems when the system load is high, per report from Zdenek
Kotala in <1250860954.1239.114.camel@localhost>; instead of calling kill
directly, have the signal handler set a flag which is checked in ServerLoop.
This way, the handler can return before being called again by a subsequent
signal sent from the autovacuum launcher.  Also, increase the sleep in the
launcher in this failure path to 1 second.

Backpatch to 8.3, which is when the signalling between autovacuum
launcher/postmaster was introduced.

Also, add a couple of ReleasePostmasterChildSlot calls in error paths; this
part backpatched to 8.4 which is when the child slot stuff was introduced.
src/backend/postmaster/autovacuum.c
src/backend/postmaster/postmaster.c