Re: Add pg_stat_autovacuum_priority
Sami Imseih <samimseih@gmail.com>
From: Sami Imseih <samimseih@gmail.com>
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>,
Robert Treat <rob@xzilla.net>, satyanarlapuram@gmail.com, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2026-04-01T23:04:36Z
Lists: pgsql-hackers
Attachments
- v7-0002-Add-elevel-parameter-to-relation_needs_vacanalyze.patch (application/octet-stream) patch v7-0002
- v7-0004-Add-pg_stat_autovacuum_priority-view.patch (application/octet-stream) patch v7-0004
- v7-0001-Always-compute-autovacuum-priority-scores.patch (application/octet-stream) patch v7-0001
- v7-0003-Refactor-autovacuum-score-computation-into-comput.patch (application/octet-stream) patch v7-0003
> I've been thinking about whether to also return whether autovacuum is > enabled in the view, i.e., AutoVacuumingActive() && av_enabled. I don't think we can rely on AutoVacuumingActive() being stable since a backend that does a SET track_counts = off for whatever reason and then calls the view will get false. av_enabled will likely be the only thing we can report. > So, I'm > currently leaning towards leaving that information out for now. I agree. > > scores->needs_vacuum = (vactuples > vacthresh); > *do_vacuum |= (av_enabled && scores->needs_vacuum); > > ... but others might find your version easier to read. yeah, for readability, I'll stick with the current. > Otherwise, 0001 looks good. > > In 0003, I think you missed renaming the last argument to > compute_autovac_score() in table_recheck_autovac(). Earlier, I did not compile with 0003 only. Fixed. > I didn't see anything else in this read-through. I'm planning to start > preparing this for commit tomorrow. Thanks! here is v7 -- 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