On Wed, May 22, 2024 at 02:24:37PM +0000, Ilyasov Ian wrote:
> I corrected my patch according to what I think
> Michael wanted. I attached the new patch to the letter.

Thanks for compiling this patch.  Yes, that's the idea.

-         qr/processing remote data for replication origin \"pg_\d+\" during 
message type "INSERT" for replication target relation "public.tbl" in 
transaction \d+, finished at ([[:xdigit:]]+\/[[:xdigit:]]+)/
+         qr/ERROR:  duplicate key.*\n.*DETAIL:.*\n.*CONTEXT:.* finished at 
([[:xdigit:]]+\/[[:xdigit:]]+)/m

There are three CONTEXT strings that could map to this context.  It
seems to me that we should keep the 'for replication target relation
"public.tbl" in transaction \d+,', before the "finished at" so as it
is possible to make a difference with the context that has a column
name and the context where there is no target relation.  That makes
the regexp longer, but it is not that bad.
--
Michael

Attachment: signature.asc
Description: PGP signature

Reply via email to