From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, "Anton A(dot) Melnikov" <a(dot)melnikov(at)postgrespro(dot)ru>, Daniel Gustafsson <daniel(at)yesql(dot)se>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Use XLOG_CONTROL_FILE macro everywhere? |
Date: | 2024-04-27 09:12:05 |
Message-ID: | 769dcaa2-acb7-49e5-a0b8-f257b91d1aed@eisentraut.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 26.04.24 22:51, Tom Lane wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> On Wed, Apr 24, 2024 at 8:04 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>>> Not sure that I would bother with a second one. But, well, why not if
>>> people want to rename it, as long as you keep compatibility.
>
>> I vote for just standardizing on XLOG_CONTROL_FILE. That name seems
>> sufficiently intuitive to me, and I'd rather have one identifier for
>> this than two. It's simpler that way.
>
> +1. Back when we did the great xlog-to-wal renaming, we explicitly
> agreed that we wouldn't change internal symbols referring to xlog.
> It might or might not be appropriate to revisit that decision,
> but I sure don't want to do it piecemeal, one symbol at a time.
>
> Also, if we did rename this one, the logical choice would be
> WAL_CONTROL_FILE not PG_CONTROL_FILE.
My reasoning was mainly that I don't see pg_control as controlling just
the WAL. But I don't feel strongly about instigating a great renaming
here or something.
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2024-04-27 12:15:36 | Re: pgsql: psql: add an optional execution-count limit to \watch. |
Previous Message | Anton A. Melnikov | 2024-04-27 08:27:01 | Re: Refactoring backend fork+exec code |