Thread: RE: Adding a '--two-phase' option to 'pg_createsubscriber' utility.
RE: Adding a '--two-phase' option to 'pg_createsubscriber' utility.
From
"Hayato Kuroda (Fujitsu)"
Date:
Dear Shubham, Thanks for the proposal! > I am writing to propose the addition of the two_phase option in > pg_createsubscriber. As discussed in [1], supporting this feature > during the development of pg_createsubscriber was planned for this > version. Yes, that was the pending item. > The attached patch has the changes for the same. There are API related comments. 01. I think the two-phase should be off by default. Because default value for CREATE SUBSCRIPTION command is off, and your patch breaks pre-existing style. 02. API style should be changed: no need to require the argument. I think it is enough to provide "enable-twophase" and "T" option which specify the two_phase to true. Best regards, Hayato Kuroda FUJITSU LIMITED
On Tue, Dec 10, 2024 at 7:42 AM Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com> wrote: > > Dear Shubham, > > Thanks for the proposal! > > > I am writing to propose the addition of the two_phase option in > > pg_createsubscriber. As discussed in [1], supporting this feature > > during the development of pg_createsubscriber was planned for this > > version. > > Yes, that was the pending item. > > > The attached patch has the changes for the same. > > There are API related comments. > > 01. > I think the two-phase should be off by default. Because default value for CREATE > SUBSCRIPTION command is off, and your patch breaks pre-existing style. > > 02. > API style should be changed: no need to require the argument. I think it is enough > to provide "enable-twophase" and "T" option which specify the two_phase to true. > I have fixed the given comments. The v2 version patch attached at [1] has the changes for the same. [1] - https://www.postgresql.org/message-id/CAHv8RjLcdmz%3D_RMwveuDdr8i7r%3D09TAwtOnFmXeaia_v2RmnYA%40mail.gmail.com Thanks and Regards, Shubham Khanna.