Re: [HACKERS] logical decoding of two-phase transactions

From: Ajin Cherian <itsajin(at)gmail(dot)com>
To: Peter Smith <smithpb2250(at)gmail(dot)com>
Cc: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>, Greg Nancarrow <gregn4422(at)gmail(dot)com>, "tanghy(dot)fnst(at)fujitsu(dot)com" <tanghy(dot)fnst(at)fujitsu(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [HACKERS] logical decoding of two-phase transactions
Date: 2021-07-01 03:48:20
Message-ID: CAFPTHDbQPR=-E+8jh7a_jbdTovYcR31Eoa_aAo6D4Wg6Lj_FhQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jun 30, 2021 at 7:47 PM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> On Wed, Jun 30, 2021 at 6:14 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> > Pushed the patch after taking care of your suggestion. Now, the next
> > step is to rebase the remaining patches and adapt some of the checks
> > to PG-15.
>
> Please find attached the latest patch set v91*
>
> Differences from v90* are:
>
> * This is the first patch set for PG15
>
> * Rebased to HEAD @ today.
>
> * Now the patch set has only 3 patches again because v90-0001,
> v90-0002 are already pushed [1]
>
> * Bumped all relevant server version checks to 150000

Adding a new patch (0004) to this patch-set that handles skipping of
empty streamed transactions. patch-0003 did not
handle empty streamed transactions. To support this, added a new flag
"sent_stream_start" to PGOutputTxnData.
Also transactions which do not have any data will not be stream
committed or stream prepared or stream aborted.
Do review and let me know if you have any comments.

regards,
Ajin Cherian
Fujitsu Australia

Attachment Content-Type Size
v92-0001-Add-support-for-prepared-transactions-to-built-i.patch application/octet-stream 144.6 KB
v92-0003-Skip-empty-transactions-for-logical-replication.patch application/octet-stream 27.4 KB
v92-0004-Skip-empty-streaming-in-progress-transaction-for.patch application/octet-stream 9.4 KB
v92-0002-Add-prepare-API-support-for-streaming-transactio.patch application/octet-stream 54.1 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amul Sul 2021-07-01 03:50:42 Re: New committers: Daniel Gustafsson and John Naylor
Previous Message Greg Nancarrow 2021-07-01 03:46:03 Re: [bug?] Missed parallel safety checks, and wrong parallel safety