]> granicus.if.org Git - postgresql/commit
Avoid useless respawining the autovacuum launcher at high speed.
authorRobert Haas <rhaas@postgresql.org>
Fri, 20 Jan 2017 20:55:45 +0000 (15:55 -0500)
committerRobert Haas <rhaas@postgresql.org>
Fri, 20 Jan 2017 21:11:45 +0000 (16:11 -0500)
commit746ba76f1e15e2a7de1bda1eab9b1b8d84875ea5
tree5a96952efe3a96c21f5f63873094edc5affece1d
parentfd081cabf7c3ce514d28e8de1a9b5c8717ea1130
Avoid useless respawining the autovacuum launcher at high speed.

When (1) autovacuum = off and (2) there's at least one database with
an XID age greater than autovacuum_freeze_max_age and (3) all tables
in that database that need vacuuming are already being processed by a
worker and (4) the autovacuum launcher is started, a kind of infinite
loop occurs.  The launcher starts a worker and immediately exits.  The
worker, finding no worker to do, immediately starts the launcher,
supposedly so that the next database can be processed.  But because
datfrozenxid for that database hasn't been advanced yet, the new
worker gets put right back into the same database as the old one,
where it once again starts the launcher and exits.  High-speed ping
pong ensues.

There are several possible ways to break the cycle; this seems like
the safest one.

Amit Khandekar (code) and Robert Haas (comments), reviewed by
Álvaro Herrera.

Discussion: http://postgr.es/m/CAJ3gD9eWejf72HKquKSzax0r+epS=nAbQKNnykkMA0E8c+rMDg@mail.gmail.com
src/backend/postmaster/autovacuum.c