MIME-Version: 1.0
References: <adB9nSsm_S0D9708@nathan>
 <CAOzEurT6RJdEErH9=8jugMy5UMak1ZPJdY4xQ1pWS4VDwuoraQ@mail.gmail.com>
In-Reply-To: 
 <CAOzEurT6RJdEErH9=8jugMy5UMak1ZPJdY4xQ1pWS4VDwuoraQ@mail.gmail.com>
From: Sami Imseih <samimseih@gmail.com>
Date: Sun, 5 Apr 2026 17:28:06 -0500
Message-ID: 
 <CAA5RZ0uSwqD8Sh5FZ8Q0QbHqcL2+jhc78ABEoeArVdU6-13U5A@mail.gmail.com>
Subject: Re: remove autoanalyze corner case
To: Shinya Kato <shinya11.kato@gmail.com>
Cc: Nathan Bossart <nathandbossart@gmail.com>,
	Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>,
 Robert Treat <rob@xzilla.net>,
	satyanarlapuram@gmail.com, pgsql-hackers <pgsql-hackers@postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Archived-At: 
 <https://www.postgresql.org/message-id/CAA5RZ0uSwqD8Sh5FZ8Q0QbHqcL2%2Bjhc78ABEoeArVdU6-13U5A%40mail.gmail.com>
Precedence: bulk

> > Well, autoanalyze only runs in this case if autovacuum is disabled via the
> > table's autovacuum_enabled reloption and _not_ disabled via the autovacuum
> > or track_counts GUCs.  I think this is pretty clearly unintentional, as I
> > can find no mention in the code, archives, or docs.  And unless I'm missing
> > something, it's completely unnecessary.  So IMHO we should just remove it.
>
> +1. I think it might be better to mention this in the documentation to
> make the behavior explicit. What do you think about stating that an
> anti-wraparound autovacuum does not perform an ANALYZE, like in the
> attached patch?

+1 to the additional documentation.

a small nit: We can just say "skipped" and not "still skipped".

--
Sami