(62) Fixed. Added assert for twophase alter check in maybe_reread_subscription(void)
(63) Fixed. Changed parse_output_parameters to disable two-phase and streaming combo
Amit 16 Mar -
https://www.postgresql.org/message-id/CAA4eK1Kwah%2BMimFMR3jPY5cSqpGFVh5zfV2g4%3DgTphaPsacoLw%40mail.gmail.com(74) Fixed. Modify comment about why not supporting combination of two-phase and streaming
(75) Fixed. Added more comments about creating slot with two-phase race conditions
(78) Skipped. Adding assert for two-phase variables getting reset, the logic has been changed, so skipping this.
(79) Changed. Reworded the comment about allowing decoding of prepared transaction (restoring iff)
(80) Fixed. Added & in the assignment for ctx->twophase, logic is also changed
(81) Fixed. Changed to conditional setting of two_phase_at only if two_phase is enabled.
(82) Fixed. Better explanation for two_phase_at variable in snapbuild.changed
(83) Skipped. The comparison in ReorderBufferFinishPrepared was not changed and it was tested and it works.
The reason it works is because even if the Prepare is filtered when two-phase is not enabled, once the tablessync is
over and the TABLES are in READY state, the apply worker and the walsender restarts, and after restart, the prepare will be
not be filtered out, but will be marked as skipped prepare and also updated in ReorderBufferRememberPrepareInfo
(87) Fixed. Added server version check before two-phase enabled startstream in ApplyWorkerMain.
(91)Fixed. Removed unused macros in reorderbuffer.h
Amit 17/Mar -
https://www.postgresql.org/message-id/CAA4eK1LNLA20ci3_qqNQv7BYRTy3HqiAsOfuieqo6tJ2GeYuJw%40mail.gmail.com(96) Fixed - Removed token for twophase in Start Replication slot, instead used the twophase options. But kept the token
in Create_Replication slot, as we gave the option for plugins to enable two-phase while creating a slot. This allows plugins without a table-synchronization phase
to handle two-phase from the start.
regards,
Ajin Cherian
Fujitsu Australia
