Re: BUG #18280: logical decoding build wrong snapshot for subtransactions

From: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
To: feichanghong <feichanghong(at)qq(dot)com>
Cc: pgsql-bugs <pgsql-bugs(at)lists(dot)postgresql(dot)org>
Subject: Re: BUG #18280: logical decoding build wrong snapshot for subtransactions
Date: 2024-01-23 11:42:34
Message-ID: CAA4eK1L7FC3o6tqnXXimxTc7wmUXw2SWQ7bs2Fc_eBK2OpFSDg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Wed, Jan 10, 2024 at 7:56 PM feichanghong <feichanghong(at)qq(dot)com> wrote:
>
> > SnapBuildXidSetCatalogChanges (introduced in 272248a) skipping the check for
> > the sub-transactions when the parent transaction has been marked as
> > containing catalog changes should be the root cause of the problem.
>
> My fix is to remove the ReorderBufferXidHasCatalogChanges condition from the SnapBuildXidSetCatalogChanges function. This approach has a false positive; the subtransactions that didn't change catalog to the snapshot are also added to the commit list. But that won't be a problem since we use snapshot built during decoding only to read system catalogs, as 272248a's commit message says.
>

Thanks, your analysis sounds correct to me. BTW, I think this should
occur only in 15 and 14 because in prior branches we didn't check if
the xact has catalog changes before marking its subxacts. From 16
onwards, we already accurately serialize the xacts and subxacts that
have made any catalog changes, so this problem shouldn't be there. Can
you please once verify the same?

--
With Regards,
Amit Kapila.

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message feichanghong 2024-01-23 12:27:59 Re: BUG #18280: logical decoding build wrong snapshot for subtransactions
Previous Message Richard Guo 2024-01-23 10:41:41 Re: BUG #18305: Unexpected error: "WindowFunc not found in subplan target lists" triggered by subqueries