Re: logicalrep_message_type throws an error - Mailing list pgsql-hackers

From Masahiko Sawada
Subject Re: logicalrep_message_type throws an error
Date
Msg-id CAD21AoBs=89deXSWzdNC8Kxk1_3Rpj9qbK49HY6-1J6ADXPXkg@mail.gmail.com
Whole thread Raw
In response to Re: logicalrep_message_type throws an error  (Amit Kapila <amit.kapila16@gmail.com>)
Responses Re: logicalrep_message_type throws an error
Re: logicalrep_message_type throws an error
List pgsql-hackers
On Thu, Jul 6, 2023 at 6:28 PM Amit Kapila <amit.kapila16@gmail.com> wrote:
>
> One point to note is that the user may also get confused if the actual
> ERROR says message type as 'X' and the context says '???'. I feel in
> this case duplicate information is better than different information.

I agree. I think it would be better to show the same string like:

ERROR:  invalid logical replication message type "??? (88)"
CONTEXT:  processing remote data for replication origin "pg_16638"
during message type "??? (88)" in transaction 796, finished at
0/1626698

Since the numerical value is important only in invalid message type
cases, how about using a format like "??? (88)" in unknown message
type cases, in both error and context messages?

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com



pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: unrecognized node type while displaying a Path due to dangling pointer
Next
From: Peter Smith
Date:
Subject: Re: doc: improve the restriction description of using indexes on REPLICA IDENTITY FULL table.