From: Álvaro Herrera <alvherre@kurilemu.de>
To: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Cc: Peter Eisentraut <peter@eisentraut.org>
Cc: Michael Paquier <michael@paquier.xyz>
Cc: Heikki Linnakangas <hlinnaka@iki.fi>
Cc: Quan Zongliang <quanzongliang@yeah.net>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: Consistently use the XLogRecPtrIsInvalid() macro
Date: Thu, 6 Nov 2025 10:06:13 +0100
Message-ID: <202511060901.sum2b5xs6dfv@alvherre.pgsql> (raw)
In-Reply-To: <aQw4n7V1luk1xnrI@ip-10-97-1-34.eu-west-3.compute.internal>
On 2025-Nov-06, Bertrand Drouvot wrote:
> Subject: [PATCH v5 1/4] Introduce XLogRecPtrIsValid() and replace
> XLogRecPtrIsInvalid() calls
> XLogRecPtrIsInvalid() is inconsistent with the affirmative form of other
> *IsValid() macros and leads to awkward double negative.
>
> This commit introduces XLogRecPtrIsValid() and replace all the
> XLogRecPtrIsInvalid() calls.
>
> It also adds a comment mentioning that new code should use XLogRecPtrIsValid()
> instead of XLogRecPtrIsInvalid() and that XLogRecPtrIsInvalid() could be
> deprecated in the future.
I think we should do this in two steps. First, introduce
XLogRecPtrIsValid(), don't use it anywhere, backpatch this one. This
would alleviate potential backpatching pains when using the new macro in
future bugfixes. Second, change calls of the old function to the new
one, no backpatch.
> From 22f02ca0618d9f2e34de8fa084127bf500d75603 Mon Sep 17 00:00:00 2001
> From: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
> Date: Mon, 3 Nov 2025 06:33:01 +0000
> Subject: [PATCH v5 2/4] Introduce PG_DEPRECATED() and deprecate
> XLogRecPtrIsInvalid()
The uppercase name looks a bit ugly. We use lowercase for other uses of
__attribute__, e.g. pg_attribute_aligned(). Also, probably add
"attribute" to the name, for consistency with those.
--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: alvherre@kurilemu.de, bertranddrouvot.pg@gmail.com, peter@eisentraut.org, michael@paquier.xyz, hlinnaka@iki.fi, quanzongliang@yeah.net, pgsql-hackers@lists.postgresql.org
Subject: Re: Consistently use the XLogRecPtrIsInvalid() macro
In-Reply-To: <202511060901.sum2b5xs6dfv@alvherre.pgsql>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox