Re: [COMMITTERS] pgsql: pg_rewind: Fix some problems when copying files >2GB.

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Robert Haas <rhaas(at)postgresql(dot)org>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: pg_rewind: Fix some problems when copying files >2GB.
Date: 2017-08-28 13:05:45
Message-ID: 20170828130545.sdajqlpr37hmmd6a@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Robert Haas wrote:
> pg_rewind: Fix some problems when copying files >2GB.

I just noticed that this broke pg_rewind translation, because of the
INT64_FORMAT marker in the translatable string. The message catalog now
has this:

msgid "received chunk for file \"%s\", offset "

for this source line:

pg_log(PG_DEBUG, "received chunk for file \"%s\", offset " INT64_FORMAT ", size %d\n",
filename, chunkoff, chunksize);

I don't think that this is terribly valuable as a translatable string,
so my preferred fix would be to have a new function pg_debug() here
where the messages are *not* marked for translations.

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2017-08-28 14:08:44 Re: [COMMITTERS] pgsql: pg_rewind: Fix some problems when copying files >2GB.
Previous Message Fabrízio de Royes Mello 2017-08-28 10:16:50 Re: pgsql: Release notes for 9.6.5, 9.5.9, 9.4.14, 9.3.19, 9.2.23.

Browse pgsql-hackers by date

  From Date Subject
Next Message Ryan Murphy 2017-08-28 13:47:17 Re: 1 test fails in make installcheck-world - database "regress_ecpg_user2" does not exist
Previous Message Tom Lane 2017-08-28 13:04:40 Re: [HACKERS] [postgresql 10 beta3] unrecognized node type: 90