Re: Berserk Autovacuum (let's save next Mandrill)
David Rowley <david.rowley@2ndquadrant.com>
From: David Rowley <david.rowley@2ndquadrant.com>
To: Darafei Komяpa Praliaskouski <me@komzpa.net>
Cc: Alvaro Herrera <alvherre@2ndquadrant.com>, PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>
Date: 2019-03-28T09:32:21Z
Lists: pgsql-hackers
On Thu, 28 Mar 2019 at 22:04, Darafei "Komяpa" Praliaskouski <me@komzpa.net> wrote: > > On Thu, Mar 28, 2019 at 2:36 AM David Rowley <david.rowley@2ndquadrant.com> wrote: >> I thought recently that it would be good to have some sort of >> pro-active auto-vacuum mode that made use of idle workers. > > Problem with "idle" is that it never happens on system that are going to wraparound on their lifetime. This has to be a part of normal database functioning. I'd say auto-vacuum is configured to run too slowly if you never have an idle worker. The chances that it happens to be running at exactly the right speed to keep up with demand must be about close to nil. > Why not select a table that has inserts, updates and deletes for autovacuum just like we do for autoanalyze, not only deletes and updates like we do now? Sounds like a good idea, although I do agree with Alvaro when he mentions that it would be good to only invoke a worker that was only going to freeze tuples and not look at the indexes. I've not looked at it, but there's a patch [1] in the current CF for that. I'd say a good course of action would be to review that then write a patch with a new bool flag in relation_needs_vacanalyze for "freezeonly" and have auto-vacuum invoke vacuum in this new freeze only mode if freezeonly is set and dovacuum is not. Any patch not in the current CF is already PG13 or beyond. Having at least a freeze only vacuum mode main ease some pain, even if it still needs to be done manually for anyone finding themselves in a similar situation as mailchimp. The idea I was mentioning was more targeted to ease the sudden rush of auto-vacuum activity when suddenly a bunch of large tables require an anti-wraparound vacuum all at once. [1] https://commitfest.postgresql.org/22/1817/ -- David Rowley http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
Commits
-
Further improve stability fix for partition_aggregate test.
- 18d85e9b8a2b 13.0 landed
-
Improve stability fix for partition_aggregate test.
- 7cb0a423f914 13.0 landed
-
Attempt to stabilize partitionwise_aggregate test
- cefb82d49e21 13.0 landed
-
Fix race condition in statext_store().
- fe3036527a1f 13.0 landed
- 6c426cd43790 12.3 landed
-
Attempt to fix unstable regression tests, take 2
- 24566b359d09 13.0 landed
-
Attempt to fix unstable regression tests
- 2dc16efedc76 13.0 landed
-
Trigger autovacuum based on number of INSERTs
- b07642dbcd8d 13.0 landed
-
Fix upper limit for vacuum_cleanup_index_scale_factor
- 4d54543efa5e 11.0 cited