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 20:55:45 +0000 (15:55 -0500)
commitc6a389792e08980e395055ac219303cf4357ba33
tree032735da70a93820b9ce325329651aeaf83c831a
parent6546ffb35db78365d9f0011d75d16625e6040437
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