From: | wenhui qiu <qiuwenhuifx(at)gmail(dot)com> |
---|---|
To: | Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> |
Cc: | Sami Imseih <samimseih(at)gmail(dot)com>, Nathan Bossart <nathandbossart(at)gmail(dot)com>, Michael Banck <mbanck(at)gmx(dot)net>, Shinya Kato <shinya11(dot)kato(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Add log_autovacuum_{vacuum|analyze}_min_duration |
Date: | 2025-06-04 01:30:25 |
Message-ID: | CAGjGUA+UXdNexKshmi24H8PVUascMsWwvouMhEaLjAeuO8UGsw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
HI
I vote log_autovacuum_{vacuum|analyze}_min_duration. Then don't remove
log_autovacuum_min_duration so easily!
On Wed, Jun 4, 2025 at 7:16 AM Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
wrote:
>
>
> On 2025/06/04 4:32, Sami Imseih wrote:
> >> On Tue, Jun 03, 2025 at 10:57:11AM +0200, Michael Banck wrote:
> >>> On Tue, Jun 03, 2025 at 05:25:40PM +0900, Shinya Kato wrote:
> >>>> I surely think adding log_autoanalyze_min_duration is simpler and
> >>>> shorter, but the reason I chose this GUC name is for consistency with
> >>>> other autovacuum parameters. Existing autovacuum parameters that have
> >>>> separate settings for vacuum and analyze operations follow the pattern
> >>>> autovacuum_{vacuum|analyze}_*.
> >>>>
> https://www.postgresql.org/docs/devel/runtime-config-vacuum.html#RUNTIME-CONFIG-AUTOVACUUM
> >>>
> >>> Right, but the GUCs that directly affect either vacuum or autovacuum
> >>> behaviour need the qualification (and then vacuum/analyze on top of
> it).
> >>> I think we have less constraints with the logging GUC and do not need
> to
> >>> mirror the behaviorial GUCs at all costs. But again, that is just my
> two
> >>> cents.
> >>
> >> I lean towards log_autovacuum_{vacuum|analyze}_min_duration. If
> >> log_autovacuum_min_duration didn't exist, that's probably the naming
> scheme
> >> we'd go with. However, I'm not sure we can get away with renaming
> >> log_autovacuum_min_duration. Presumably we'd need to at least keep it
> >> around as a backward-compatibility GUC, and its behavior would probably
> >> change, too
> >
> > I think deprecating a GUC like log_autovacuum_min_duration would be quite
> > difficult.
>
> Also deprecating the log_autovacuum_min_duration reloption might be tricky.
> If we remove support for it in v19, how should pg_dump handle tables with
> this option set from older versions? Should it translate it into both
> log_autovacuum_vacuum_min_duration and log_autovacuum_analyze_min_duration
> during dump? Would pg_upgrade run into the same issue?
>
> Regards,
>
> --
> Fujii Masao
> NTT DATA Japan Corporation
>
>
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Xuneng Zhou | 2025-06-04 03:11:17 | Re: Proposal: Limitations of palloc inside checkpointer |
Previous Message | Masahiko Sawada | 2025-06-04 01:10:44 | Re: POC: enable logical decoding when wal_level = 'replica' without a server restart |