pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Álvaro Herrera <alvherre@kurilemu.de>
To: Michael Paquier <michael@paquier.xyz>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: Translation of the NextOID message in pg_controldata
Date: Mon, 14 Sep 2026 08:41:59 +0200
Message-ID: <aqeO1OU7uZjhioAq@alvherre.pgsql> (raw)
In-Reply-To: <aqeNcd4403BGVZqc@paquier.xyz>

On 2026-Sep-14, Michael Paquier wrote:

> The magic solution I can think of is to remove the markers from the
> translatable strings, and replace them with a set of %s, as of the
> attached patch, then use a pre-built string that itself uses
> OID8_FORMAT.  I am pretty sure we have used this method in other
> places of the tree, but I cannot pinpoint where, on top of my mind.

Yeah, we used to do that for long long, until we got rid of it using the
%z format instead.  I was happy about no longer having that coding
pattern TBH.

I think this change would break what I proposed in
https://postgr.es/m/202601310924.yoik5n3blgt4@alvherre.pgsql
(FTR I decided to heed Peter's advice and not implement it in the way
proposed there, but instead build some generic facility that can also be
used elsewhere.)

I'm not sure I understand why we care that we hardcode that Oid8 is the
same as uint64.  It's never been a problem that Oid is the same as uin32 ...

-- 
Álvaro Herrera        Breisgau, Deutschland  —  https://www.EnterpriseDB.com/
"No tengo por qué estar de acuerdo con lo que pienso"
                             (Carlos Caszeli)





view thread (10+ messages)  latest in thread

Message-ID: <aqeO1OU7uZjhioAq@alvherre.pgsql>
Permalink:  ../aqeO1OU7uZjhioAq@alvherre.pgsql/
Also on:    postgresql.org/message-id/aqeO1OU7uZjhioAq@alvherre.pgsql

reply

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, michael@paquier.xyz, tgl@sss.pgh.pa.us, horikyota.ntt@gmail.com, pgsql-hackers@lists.postgresql.org
  Subject: Re: Translation of the NextOID message in pg_controldata
  In-Reply-To: <aqeO1OU7uZjhioAq@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