Re: Berserk Autovacuum (let's save next Mandrill)
Andres Freund <andres@anarazel.de>
From: Andres Freund <andres@anarazel.de>
To: Laurenz Albe <laurenz.albe@cybertec.at>
Cc: Justin Pryzby <pryzby@telsasoft.com>, Masahiko Sawada <masahiko.sawada@2ndquadrant.com>, David Rowley <dgrowleyml@gmail.com>, Tomas Vondra <tomas.vondra@2ndquadrant.com>, Amit Kapila <amit.kapila16@gmail.com>, Masahiko Sawada <sawada.mshk@gmail.com>, Darafei Komяpa Praliaskouski <me@komzpa.net>, PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>, Alvaro Herrera <alvherre@2ndquadrant.com>, Michael Banck <mbanck@gmx.net>
Date: 2020-03-18T01:02:54Z
Lists: pgsql-hackers
Hi, On 2020-03-17 20:42:07 +0100, Laurenz Albe wrote: > > I think Andres was thinking this would maybe be an optimization independent of > > is_insert_only (?) > > I wasn't sure. I'm not sure myself - but I'm doubtful that using a 0 min age by default will be ok. I was trying to say (in a later email) that I think it might be a good compromise to opportunistically freeze if we're dirtying the page anyway, but not optimize WAL emission etc. That's a pretty simple change, and it'd address a lot of the potential performance regressions, while still freezing for the "first" vacuum in insert only workloads. > Add "autovacuum_vacuum_insert_threshold" and > "autovacuum_vacuum_insert_scale_factor" GUC and reloption. > The default value for the threshold is 10000000. > The scale factor defaults to 0, which means that it is > effectively disabled, but it offers some flexibility > to tune the feature similar to other autovacuum knobs. I don't think a default scale factor of 0 is going to be ok. For large-ish tables this will basically cause permanent vacuums. And it'll sometimes trigger for tables that actually coped well so far. 10 million rows could be a few seconds, not more. I don't think that the argument that otherwise a table might not get vacuumed before autovacuum_freeze_max_age is convincing enough. a) if that's indeed the argument, we should increase the default autovacuum_freeze_max_age - now that there's insert triggered vacuums, the main argument against that from before isn't valid anymore. b) there's not really a good arguments for vacuuming more often than autovacuum_freeze_max_age for such tables. It'll not be not frequent enough to allow IOS for new data, and you're not preventing anti-wraparound vacuums from happening. Greetings, Andres Freund
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