Re: Review: Extra Daemons / bgworker
Alvaro Herrera <alvherre@2ndquadrant.com>
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Simon Riggs <simon@2ndQuadrant.com>
Cc: Markus Wanner <markus@bluegap.ch>, kaigai@kaigai.gr.jp, PostgreSQL-development Hackers <pgsql-hackers@postgresql.org>
Date: 2012-12-03T15:44:46Z
Lists: pgsql-hackers
Simon Riggs wrote: > On 3 December 2012 15:17, Markus Wanner <markus@bluegap.ch> wrote: > > The only process that currently starts background workers ... ehm ... > > autovacuum workers is the autovacuum launcher. It uses the above > > Postmaster Signal in autovacuum.c:do_start_autovacuum_worker() to have > > the postmaster launch bg/autovac workers on demand. > > My understanding was that the patch keep autovac workers and > background workers separate at this point. That is correct. > Is there anything to be gained *now* from merging those two concepts? > I saw that as refactoring that can occur once we are happy it should > take place, but isn't necessary. IMO it's a net loss in robustness of the autovac implementation. -- Álvaro Herrera http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services