Re: [BUG] Logical replica crash if there was an error in a function. - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [BUG] Logical replica crash if there was an error in a function.
Date
Msg-id 20221010100645.pfula2bpv56a6txm@alvherre.pgsql
Whole thread Raw
In response to Re: [BUG] Logical replica crash if there was an error in a function.  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On 2022-Sep-24, Tom Lane wrote:

> "Anton A. Melnikov" <aamelnikov@inbox.ru> writes:
> > [ v4-0001-Fix-logical-replica-assert-on-func-error.patch ]
> 
> I took a quick look at this.  I think you're solving the
> problem in the wrong place.  The real issue is why are
> we not setting up ActivePortal correctly when running
> user-defined code in a logrep worker?  There is other code
> that expects that to be set, eg EnsurePortalSnapshotExists.

Right ... mostly, the logical replication *does* attempt to set up a
transaction and active snapshot when replaying actions (c.f.
begin_replication_step()).  Is this firing at an inappropriate time,
perhaps?

-- 
Álvaro Herrera         PostgreSQL Developer  —  https://www.EnterpriseDB.com/



pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: make_ctags: use -I option to ignore pg_node_attr macro
Next
From: Robert Haas
Date:
Subject: Re: problems with making relfilenodes 56-bits