Re: Add pg_stat_autovacuum_priority
Sami Imseih <samimseih@gmail.com>
From: Sami Imseih <samimseih@gmail.com>
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Alexander Lakhin <exclusion@gmail.com>, Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>,
Robert Treat <rob@xzilla.net>, satyanarlapuram@gmail.com, pgsql-hackers <pgsql-hackers@postgresql.org>, tndrwang@gmail.com
Date: 2026-04-08T18:28:06Z
Lists: pgsql-hackers
> diff --git a/src/backend/postmaster/autovacuum.c b/src/backend/postmaster/autovacuum.c > index bd626a16363..6d4a34257fb 100644 > --- a/src/backend/postmaster/autovacuum.c > +++ b/src/backend/postmaster/autovacuum.c > @@ -3327,7 +3327,15 @@ relation_needs_vacanalyze(Oid relid, > anltuples, anlthresh, scores->anl, > scores->xid, scores->mxid); > > - pfree(tabentry); > + /* > + * Avoid leaking pgstat entries until the end of autovacuum. Elsewhere, > + * we let the commit/abort machinery take care of freeing it. While > + * autovacuum workers set stats_fetch_consistency to "none", it might be > + * set to a different value in other processes, so we can't safely free it > + * here for them. > + */ > + if (AmAutoVacuumWorkerProcess()) > + pfree(tabentry); > } This works too, but v1-0001 is more generalized and we don't have to care about who the caller is when deciding to free or not. -- Sami
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Fix double-free in pg_stat_autovacuum_scores.
- 71ff232a5bc4 19 (unreleased) landed
-
Add LOG_NEVER error level code.
- 60165db6e1f2 19 (unreleased) landed
-
Add pg_stat_autovacuum_scores system view.
- 87f61f0c8280 19 (unreleased) landed
-
Remove recheck_relation_needs_vacanalyze().
- 775fe51daaef 19 (unreleased) landed
-
Add elevel parameter to relation_needs_vacanalyze().
- 01876ace1369 19 (unreleased) landed
-
Teach relation_needs_vacanalyze() to always compute scores.
- 53b8ca6881a1 19 (unreleased) landed
-
Refactor relation_needs_vacanalyze().
- 8261ee24fe33 19 (unreleased) landed
-
Add rudimentary table prioritization to autovacuum.
- d7965d65fc5b 19 (unreleased) cited
-
Fix per-relation memory leakage in autovacuum.
- 02502c1bca54 18.0 cited
-
Fix recently-understood problems with handling of XID freezing, particularly
- 48188e1621bb 8.2.0 cited