On Wed, Nov 12, 2025 at 06:09:40PM +0100, Álvaro Herrera wrote:
> The comment you cite appears more or less in the same place in
> autovacuum-6.patch (the routine was called test_rel_for_autovac there),
> but I think what it really is about, is the AVOperUpdateCounts operation
> that was in autovacuum-5.patch. Note that that older patch was
> introducing a new pg_autovacuum catalog which kept track of tuple counts
> since last analyze/vacuum, and those counts needed to be updated in case
> pgstats was reset. But by patch autovacuum-6.patch this had disappeared
> (the counts were now in pgstats, per review from Tom), so we didn't need
> to reset any counters, and the AVOperUpdateCounts operation went away.
>
> It's quite possible that, having removed the "counter reset" operation
> from the proposed patch and reduced the comment to what we have today,
> we promptly forgot about the whole thing. So the comment is indeed a
> relic that can be removed.
I see, thanks for looking. I've committed the patch.
--
nathan
