Re: BUG #18369: logical decoding core on AssertTXNLsnOrder()

Alexander Lakhin <exclusion@gmail.com>

From: Alexander Lakhin <exclusion@gmail.com>

To: "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>, ocean_li_996 <ocean_li_996@163.com>

Cc: "pgsql-bugs@lists.postgresql.org" <pgsql-bugs@lists.postgresql.org>, Amit Kapila <amit.kapila16@gmail.com>

Date: 2024-03-05T04:00:00Z

Lists: pgsql-bugs

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Fix catalog lookup due to wrong snapshot for subtransactions during decoding.
- 6b77048e571d 14.11 cited

Hello Kuroda-san,

04.03.2024 15:52, Hayato Kuroda (Fujitsu) wrote:
>
> Dear Alexander,
>
> I also ran your script after applying patches on REL_14_STABLE, but I got another failure.
>
> I have not analyzed yet so I share the result as-is.
>
> Was I missing something?
>
>
> ```
>
> #0 0x00007f4f033de387 in raise () from /lib64/libc.so.6
>
> #1 0x00007f4f033dfa78 in abort () from /lib64/libc.so.6
>
> #2 0x0000000000b08152 in ExceptionalCondition (conditionName=0xcc8228 "prev_first_lsn < cur_txn->first_lsn",
>
>     errorType=0xcc8024 "FailedAssertion", fileName=0xcc8070 "reorderbuffer.c", lineNumber=916) at assert.c:69
>
> #3 0x00000000008e01ad in AssertTXNLsnOrder (rb=0x19a95c0) at reorderbuffer.c:916
>
> #4 0x00000000008dfae8 in ReorderBufferTXNByXid (rb=0x19a95c0, xid=1753, create=true, is_new=0x0, lsn=37430336,
>
>     create_as_top=true) at reorderbuffer.c:669
>
>

Yes, you're right. That script produces another failure as I mentioned
upthread, but still in AssertTXNLsnOrder().
I had thought that the fix should cover that case too, but then I found out
that the defect in question is rather new, so maybe the more focused fix is
really preferable.

Best regards,
Alexander