On Fri, 17 Apr 2026 at 09:25, Amit Kapila <[email protected]> wrote: > > On Thu, Apr 16, 2026 at 11:16 AM David G. Johnston > <[email protected]> wrote: > > > > On Wednesday, April 15, 2026, Amit Kapila <[email protected]> wrote: > >> > >> > >> So, how about: "All slots on the old cluster must be usable, i.e., > >> their pg_replication_slots.conflicting is false."? > > > > > > That works. > > > > Then we’d also change: > > > > The new cluster must not have permanent logical slots, i.e., there must be > > no slots where pg_replication_slots.temporary is false. > > > > To be: > > > > “The new cluster is only permitted to have temporary logical slots, i.e., > > ones where pg_replication_slots.temporary is true.” > > > > What you wrote is correct but it is better to keep what is not allowed > for this case. Based on that, how about the following: > "The new cluster must not have any permanent logical slots; i.e., any > existing logical slots must have pg_replication_slots.temporary set to > true."
Your version seems better, the attached v2 version patch has the changes for the same. Regards, Vignesh
v2-0001-Improve-the-wording-of-logical-slot-prerequisites.patch
Description: Binary data
