Re: Autovacuum fails to keep visibility map up-to-date in mostly-insert-only-tables

Andres Freund <andres@2ndquadrant.com> writes:
> There's rub here though. We unconditionally do:
>     /* Do post-vacuum cleanup and statistics update for each index */
>     for (i = 0; i < nindexes; i++)
>         lazy_cleanup_index(Irel[i], indstats[i], vacrelstats);

> and that's not particularly cheap. Maybe we should make that conditional
> when there's been no lazy_vacuum_index/heap calls at all?

Absolutely not.  If the cleanup step is skippable, it would be the
province of the index AM to make that decision.
        regards, tom lane