Re: Use XLOG_CONTROL_FILE macro everywhere?

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.

In response to

Browse pgsql-hackers by date

  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