Hi all,
I don't think we really need to have allow_overwrite.
It is not possible to create logical slots on standby with failover=true, therefore we can safely rely on failover being true to understand that at some point this node was a primary and that this slot is supposed to be synced.
Please see the patch attached.
Regards,
--
Alexander Kukushkin