On Tue, Jul 23, 2024 at 5:10 PM Amit Kapila <amit.kapila16@gmail.com> wrote:
>
> One minor comment:
> - if the table copy is finished (See <xref
> linkend="catalog-pg-subscription-rel"/>).
> + On the subscriber node, use the following SQL to identify which main
> + slots should be synced to the standby that we plan to promote. This query
>
> Shall we refer to these slots as replication slots instead of main
> slots in the above sentence? We don't have a main slot terminology at
> other places, so it would be better not to introduce a new one. I know
> that it was introduced in the original commit but it seems better to
> change if we agree.
Yes, it makes sense. Please find the patch with this change.
thanks
SHveta