From: | Melanie Plageman <melanieplageman(at)gmail(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Daniel Gustafsson <daniel(at)yesql(dot)se>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: pgsql: Consolidate docs for vacuum-related GUCs in new subsection |
Date: | 2025-01-13 22:35:26 |
Message-ID: | CAAKRu_bn7wXa2pwrD-1EOM=dZOW9oNj+cgpALrtzc4k-YbwXJA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Mon, Jan 13, 2025 at 3:46 PM Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote:
>
> On 2025-Jan-13, Melanie Plageman wrote:
>
> > Since I didn't hear back about this and I don't see an obvious
> > alternative reorganization in guc_tables.c, I plan to just push the
> > attached patch that updates only postgresql.conf.sample.
>
> Apologies, I was very unclear -- I didn't want to talk about the
> ordering of entries in the code, but the categorization. See the
> config_group_names list in guc_tables.c, which defines some groups.
> Each setting belongs into a group, and those groups correspond to what
> the sample config file lists as section/subsection titles and to the
> grouping in the docs. Also, this categorization affects how the entries
> are listed in the pg_settings view and in "postgres --describe-config",
> which feed from the same tables.
Oh dear, I had no idea that these categories existed. I suppose I
never paid attention to the category column in pg_settings nor used
--describe-config. Attached is a patch to fix this. I checked both
pg_settings and --describe-config output, and it seems to work.
I'm quite sorry about the extra noise this is causing (especially for
people with patch sets requiring rebasing).
> Perhaps with your changes (assuming I read your commit message right),
> we need new groups:
> VACUUMING
> VACUUMING_FREEZING
> VACUUMING_AUTOVACUUM
I've gone with VACUUM_AUTOVACUUM, VACUUM_COST_DELAY, and
VACUUM_FREEZING, but I am open to feedback.
- Melanie
Attachment | Content-Type | Size |
---|---|---|
v1-0001-Synchronize-guc_tables.c-categories-with-vacuum-d.patch | application/octet-stream | 10.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Alena Rybakina | 2025-01-13 23:51:47 | Re: pgsql: Consolidate docs for vacuum-related GUCs in new subsection |
Previous Message | Jeff Davis | 2025-01-13 22:19:11 | pgsql: Revert "TupleHashTable: store additional data along with tuple." |
From | Date | Subject | |
---|---|---|---|
Next Message | Alena Rybakina | 2025-01-13 22:37:17 | Re: Eagerly scan all-visible pages to amortize aggressive vacuum |
Previous Message | Andres Freund | 2025-01-13 22:33:41 | Re: Reorder shutdown sequence, to flush pgstats later |