From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Jeff Davis <pgsql(at)j-davis(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Amit Kapila <akapila(at)postgresql(dot)org>, Ajin Cherian <itsajin(at)gmail(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com> |
Subject: | Re: Decoding of two-phase xacts missing from CREATE_REPLICATION_SLOT command |
Date: | 2021-06-03 03:59:28 |
Message-ID: | CAA4eK1++O3zop3Gn=-8zywJRkc9D+nE87neORbJT3q8cZAgrRg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Jun 3, 2021 at 4:48 AM Jeff Davis <pgsql(at)j-davis(dot)com> wrote:
>
> Commit 19890a064 changed pg_create_logical_replication_slot() to allow
> decoding of two-phase transactions, but did not extend the
> CREATE_REPLICATION_SLOT command to support it. Strangely, it does
> extend the CreateReplicationSlotCmd struct to add a "two_phase" field,
> but doesn't set it anywhere.
>
> There were patches[1] from around the time of the commit to support
> CREATE_REPLICATION_SLOT as well.
>
> Is there a reason to support two-phase decoding, but not with the
> replication protocol? If so, why change the CreateReplicationSlotCmd
> structure as though we will support it?
>
The idea is to support two_phase via protocol with a subscriber-side
work where we can test it as well. The code to support it via
replication protocol is present in the first patch of subscriber-side
work [1] which uses that code as well. Basically, we don't have a good
way to test it without subscriber-side work so decided to postpone it
till the corresponding work is done. I think we can remove the change
in CreateReplicationSlotCmd, that is a leftover. If we have to support
it via protocol, then at the minimum, we need to enhance
pg_recvlogical so that the same can be tested.
--
With Regards,
Amit Kapila.
From | Date | Subject | |
---|---|---|---|
Next Message | vignesh C | 2021-06-03 04:08:37 | Re: locking [user] catalog tables vs 2pc vs logical rep |
Previous Message | David Rowley | 2021-06-03 03:51:48 | Re: Fixup some appendStringInfo and appendPQExpBuffer calls |