]> granicus.if.org Git - postgresql/commit
Allow multiple bgworkers to be launched per postmaster iteration.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 26 Apr 2017 20:17:29 +0000 (16:17 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 26 Apr 2017 20:17:34 +0000 (16:17 -0400)
commitaa1351f1eec4adae39be59ce9a21410f9dd42118
tree023e2310053571e80fb0844e230441a544b4bdf2
parentfda4fec578505a624e4c0f0244816ae5bc10a3d1
Allow multiple bgworkers to be launched per postmaster iteration.

Previously, maybe_start_bgworker() would launch at most one bgworker
process per call, on the grounds that the postmaster might otherwise
neglect its other duties for too long.  However, that seems overly
conservative, especially since bad effects only become obvious when
many hundreds of bgworkers need to be launched at once.  On the other
side of the coin is that the existing logic could result in substantial
delay of bgworker launches, because ServerLoop isn't guaranteed to
iterate immediately after a signal arrives.  (My attempt to fix that
by using pselect(2) encountered too many portability question marks,
and in any case could not help on platforms without pselect().)
One could also question the wisdom of using an O(N^2) processing
method if the system is intended to support so many bgworkers.

As a compromise, allow that function to launch up to 100 bgworkers
per call (and in consequence, rename it to maybe_start_bgworkers).
This will allow any normal parallel-query request for workers
to be satisfied immediately during sigusr1_handler, avoiding the
question of whether ServerLoop will be able to launch more promptly.

There is talk of rewriting the postmaster to use a WaitEventSet to
avoid the signal-response-delay problem, but I'd argue that this change
should be kept even after that happens (if it ever does).

Backpatch to 9.6 where parallel query was added.  The issue exists
before that, but previous uses of bgworkers typically aren't as
sensitive to how quickly they get launched.

Discussion: https://postgr.es/m/4707.1493221358@sss.pgh.pa.us
src/backend/postmaster/postmaster.c