On Mon, Jan 5, 2026 at 9:46 AM Hayato Kuroda (Fujitsu)
<kuroda.hayato@fujitsu.com> wrote:
>
> Dear Daniil, Chao,
>
> I was the main author of 1462aad2. It is enough to remove outdated comments atop
> the definition. In other words, your patch looks good to me.
>
> If needed, we can also notify developers that the two-phase option should not be
> altered while decoding WAL records. In logical replication, we ensure that the
> subscription is disabled and there are no apply workers. However, I don't think
> such comments can be atop the ReplicationSlotCreate(). Maybe around
> ReplicationSlotAlter(), but it may be out of scope of the initial motivation.
>
I think it is better if we add some comments atop
ReplicationSlotAlter() as you are suggesting. What do you think of the
attached?
--
With Regards,
Amit Kapila.