RE: Fix slot synchronization with two_phase decoding enabled

On Tue, Apr 1, 2025 at 2:09 PM Amit Kapila wrote:

>
> On Mon, Mar 31, 2025 at 5:0 PM Zhijie Hou (Fujitsu)
> <houzj(dot)fnst(at)fujitsu(dot)com> wrote:
> >
> > Thanks for the comments, they have been addressed in V2.
> >
>
> In the PG17 patch, we only have a check preventing failover and
> two_phase in CreateSubscription. Don't we need a similar check for AlterSubscription?
>
> Apart from the above, I have a few suggestions for changes in docs,
> error messages, and comments for both versions. See attached.

Thanks for the comments.

Here is the V3 patch set which addressed all the comments.

I also added a testcase for ALTER SUB in 0002 as suggested by
Kuroda-san off-list.

Additionally, I found that a test failed in PG17 following this
patch because it involved creating a subscription with both failover and
two-phase enabled. Since that test was designed to test the upgrade of
replication slots and is not directly related to failover functionality, I
decided to disable the failover option for test case.

Best Regards,
Hou zj