Redundant errdetail prefix "The error was:" in some logical replication messages - Mailing list pgsql-hackers

From Peter Smith
Subject Redundant errdetail prefix "The error was:" in some logical replication messages
Date
Msg-id CAHut+Ps-Qv2yQceCwobQDP0aJOkfDzRFrOaR6+2Op2K=WHGeWg@mail.gmail.com
Whole thread Raw
Responses Re: Redundant errdetail prefix "The error was:" in some logical replication messages  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
There are a couple of error messages within the logical replication
code where the errdetail text includes a prefix of "The error was:"

I could not find any other examples in all the PG src which have a
similar errdetail prefix like this.

The text seems not only redundant, but "The error was: ERROR: " also
looks a bit peculiar.

e.g.
------
2021-03-30 14:17:37.567 AEDT [22317] ERROR:  could not drop the
replication slot "tap_sub" on publisher
2021-03-30 14:17:37.567 AEDT [22317] DETAIL:  The error was: ERROR:
replication slot "tap_sub" does not exist
2021-03-30 14:17:37.567 AEDT [22317] STATEMENT:  DROP SUBSCRIPTION tap_sub;
------

PSA a small patch which simply removes this prefix "The error was:"

------
Kind Regards,
Peter Smith.
Fujitsu Australia

Attachment

pgsql-hackers by date:

Previous
From: Etsuro Fujita
Date:
Subject: Re: Obsolete comment in postgres_fdw.c
Next
From: vignesh C
Date:
Subject: Re: Replication slot stats misgivings