From: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> |
---|---|
To: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: show "aggressive" or not in autovacuum logs |
Date: | 2017-03-31 09:20:23 |
Message-ID: | CAD21AoBN-Hsih7HEP8ey6NUwQgwYj0O7eMuaRtas-6+CyafOgA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Mar 29, 2017 at 12:46 PM, Kyotaro HORIGUCHI
<horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> wrote:
> Hello, it would be too late but I'd like to propose this because
> this cannot be back-patched.
>
>
> In autovacuum logs, "%u skipped frozen" shows the number of pages
> skipped by ALL_FROZEN only in aggressive vacuum.
>
> So users cannot tell whether '0 skipped-frozen' means a
> non-agressive vacuum or no frozen-pages in an agressive vacuum.
>
> I think it is nice to have an indication whether the scan was
> "agressive" or not in log output.
Good idea. I also was thinking about this.
> Like this,
>
>> LOG: automatic aggressive vacuum of table "template1.pg_catalog.pg_statistic": index scans: 0
>
> "0 skipped frozen" is uesless in non-aggressive vacuum but
> removing it would be too-much. Inserting "aggressive" reduces
> machine-readability so it might be better in another place. The
> attached patch does the following.
>
>> LOG: automatic vacuum of table "postgres.public.pgbench_branches": mode: normal, index scans: 0
>> LOG: automatic vacuum of table "postgres.public.pgbench_branches": mode: aggressive, index scans: 0
>
Should we add this even to the manual vacuum verbose message?
Regards,
--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Masahiko Sawada | 2017-03-31 09:32:56 | Re: ANALYZE command progress checker |
Previous Message | Ashutosh Bapat | 2017-03-31 09:11:40 | Re: Foreign tables don't enforce the partition constraint |