Re: Minimal logical decoding on standbys

On 2023-Apr-03, Drouvot, Bertrand wrote:

> +/*
> + * Report terminating or conflicting message.
> + *
> + * For both, logical conflict on standby and obsolete slot are handled.
> + */
> +static void
> +ReportTerminationInvalidation(bool terminating, bool islogical, int pid,
> +                              NameData slotname, TransactionId *xid,
> +                              XLogRecPtr restart_lsn, XLogRecPtr oldestLSN)
> +{

> +    if (terminating)
> +        appendStringInfo(&err_msg, _("terminating process %d to release replication slot \"%s\""),
> +                         pid,
> +                         NameStr(slotname));
> +    else
> +        appendStringInfo(&err_msg, _("invalidating"));
> +
> +    if (islogical)
> +    {
> +        if (terminating)
> +            appendStringInfo(&err_msg, _(" because it conflicts with recovery"));

You can't build the strings this way, because it's not possible to put
the strings into the translation machinery.  You need to write full
strings for each separate case instead, without appending other string
parts later.

Thanks

-- 
Álvaro Herrera               48°01'N 7°57'E  —  https://www.EnterpriseDB.com/
"Hay quien adquiere la mala costumbre de ser infeliz" (M. A. Evans)