On 2025/06/07 0:13, Robert Treat wrote:
On Fri, Jun 6, 2025 at 9:57 AM David G. Johnston
<david.g.johns...@gmail.com> wrote:
On Friday, June 6, 2025, Fujii Masao <masao.fu...@oss.nttdata.com> wrote:

Hi,

Since last_vacuum and vacuum_count in pg_stat_all_tables explicitly mention
that they don't include VACUUM FULL ("not counting VACUUM FULL"), I think
we should add the same clarification to the description of total_vacuum_time.
This field also excludes VACUUM FULL, and without this note, users might
mistakenly think the time spent on VACUUM FULL is included. Thought?

         <structfield>total_vacuum_time</structfield> <type>double 
precision</type>
        </para>
        <para>
-       Total time this table has been manually vacuumed, in milliseconds.
+       Total time this table has been manually vacuumed, in milliseconds
+       (not counting <command>VACUUM FULL</command>).
         (This includes the time spent sleeping due to cost-based delays.)
        </para></entry>
       </row>


  Makes sense.  Our naming this table rewrite vacuum full does confuse people 
into thinking it is related to vacuum.


+1 for this change,

Thanks both for the review!


but I think we should also update
n_ins_since_vacuum as well, no?

I didn't update n_ins_since_vacuum since it's mainly used by autovacuum rather
than end users, and there haven't been any complaints about the current
description so far. That said, I don't have a strong opinion either way,
so I'm fine with making the change if others think it's worthwhile.

Regards,

--
Fujii Masao
NTT DATA Japan Corporation



Reply via email to