pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Markus Wanner <markus@bluegap.ch>
To: Amit Kapila <amit.kapila16@gmail.com>
To: Ajin Cherian <itsajin@gmail.com>
Cc: Masahiko Sawada <sawada.mshk@gmail.com>
Cc: Peter Smith <smithpb2250@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: [HACKERS] logical decoding of two-phase transactions
Date: Mon, 22 Feb 2021 18:34:13 +0100
Message-ID: <dbec82e2-dbd7-95a2-c6b6-e488cbbdf853@bluegap.ch> (raw)
In-Reply-To: <CAA4eK1KC_ZRKCMdeoWrVPtK_2=UM9715YwuVhO0sAnhqpe75jg@mail.gmail.com>
References: <CAMGcDxeqEpWj3fTXwqhSwBdXd2RS9jzwWscO-XbeCfso6ts3+Q@mail.gmail.com>
	<CAA4eK1+AspMGJEyoh4bc-wJ3fqYtFfAf375xOXBLJQKf8WCTQQ@mail.gmail.com>
	<CAA4eK1Jny30bcYZ7yQdaKu16ruw3J891vbagnOFChuqxCThwYA@mail.gmail.com>
	<CAFPTHDaZGYTGhwwaOwPHdanpz0qH8u9z49QEG93BTWr66QnpTg@mail.gmail.com>
	<CAA4eK1L3p4z+9wtK77MbdpkagR4GS2Y3r1Je7ZEvLQVF9GArfg@mail.gmail.com>
	<CAFPTHDYnRKDvzgDxoMn_CKqXA-D0MtrbyJvfvjBsO4G=UHDXkg@mail.gmail.com>
	<CAA4eK1KAJafQ8D5TnoL9RMrg5OMaCXiftUbc5z57=2dSr84cWw@mail.gmail.com>
	<CAFPTHDZMJC1-cNFk32xoVYGq15hp+cF4=MMfuwxRt7N2eSoJSA@mail.gmail.com>
	<CAD21AoA9n6qmLH2phB+rwSUgx+AzWYNj-A7M4dq4ZZUBTCt0nw@mail.gmail.com>
	<CAFPTHDb_wDLSQixranv9Gy1jspzwbcpHFjPugizs_wH+cXypDg@mail.gmail.com>
	<CAA4eK1Luw0c6B-WuO5v2EhiSrztgvSS_eo+7y5rYR622scMu2A@mail.gmail.com>
	<CAA4eK1J7TiOQSiDq9MS_4k7zr4Jca7SnO8Vy3L4wtc7t1X9zsQ@mail.gmail.com>
	<CAFPTHDbBmCb2DNRozDrkQ42=4Dam5XFjUnfZ9Ca--AVN=7CT7A@mail.gmail.com>
	<CAA4eK1KC_ZRKCMdeoWrVPtK_2=UM9715YwuVhO0sAnhqpe75jg@mail.gmail.com>

Hello Amit,

On 04.01.21 09:18, Amit Kapila wrote:
> Thanks, I have pushed the 0001* patch after making the above and a few
> other cosmetic modifications.

That commit added the following snippet to the top of 
ReorderBufferFinishPrepared:

   txn = ReorderBufferTXNByXid(rb, xid, true, NULL, commit_lsn, false);

   /* unknown transaction, nothing to do */
   if (txn == NULL)
       return;

Passing true for the create argument seems like an oversight.  I think 
this should pass false and not ever (have to) create a ReorderBufferTXN 
entry.

Regards

Markus





view thread (644+ messages)  latest in thread

Message-ID: <dbec82e2-dbd7-95a2-c6b6-e488cbbdf853@bluegap.ch>
Permalink:  ../dbec82e2-dbd7-95a2-c6b6-e488cbbdf853@bluegap.ch/
Also on:    postgresql.org/message-id/dbec82e2-dbd7-95a2-c6b6-e488cbbdf853@bluegap.ch

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: markus@bluegap.ch, amit.kapila16@gmail.com, itsajin@gmail.com, sawada.mshk@gmail.com, smithpb2250@gmail.com
  Subject: Re: [HACKERS] logical decoding of two-phase transactions
  In-Reply-To: <dbec82e2-dbd7-95a2-c6b6-e488cbbdf853@bluegap.ch>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox