MIME-Version: 1.0
References: <177609278737.403059.14174275013090471947@wrigleys.postgresql.org>
 <CAKFQuwZ86BE49-gzFunuu=u1ieT+T6q900=1Cx=vuO9Aqv9kcg@mail.gmail.com>
 <CAA4eK1LR_MjZuV31r-MOqno9-Kr0mA9zas2cxZtE=+X3RUYXbw@mail.gmail.com>
 <CAKFQuwbHjB6EEdp6mmaUt4bCTYrYUdvEr44vVqhdT0OD4Z+3Qg@mail.gmail.com>
 <CAA4eK1+dR-sEGM+oC7H_-7PrBJBcdM2o5-eS6JrkMBJ0FOA54Q@mail.gmail.com>
 <CAKFQuwaWUuXJYZSMKtiXFT=kFK-sFg+pSwK0MUN-MBEEobkfBg@mail.gmail.com>
 <CAA4eK1LWNPW7bTY+0=L183rCyqUmq6MuFkgTE8QwHeT6nkDmpg@mail.gmail.com>
In-Reply-To: 
 <CAA4eK1LWNPW7bTY+0=L183rCyqUmq6MuFkgTE8QwHeT6nkDmpg@mail.gmail.com>
From: vignesh C <vignesh21@gmail.com>
Date: Fri, 17 Apr 2026 11:02:16 +0530
Message-ID: 
 <CALDaNm1VOrMXHHPxPmaaBvLN43L2=E5d=KLLUSdiMn8SOfj1VQ@mail.gmail.com>
Subject: Re: Logical Replication upgrade
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: "David G. Johnston" <david.g.johnston@gmail.com>,
	"mimidatabase@gmail.com" <mimidatabase@gmail.com>,
	"pgsql-docs@lists.postgresql.org" <pgsql-docs@lists.postgresql.org>
Content-Type: multipart/mixed; boundary="00000000000050c232064fa14671"
Archived-At: 
 <https://www.postgresql.org/message-id/CALDaNm1VOrMXHHPxPmaaBvLN43L2%3DE5d%3DKLLUSdiMn8SOfj1VQ%40mail.gmail.com>
Precedence: bulk

--00000000000050c232064fa14671
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Fri, 17 Apr 2026 at 09:25, Amit Kapila <amit.kapila16@gmail.com> wrote:
>
> On Thu, Apr 16, 2026 at 11:16=E2=80=AFAM David G. Johnston
> <david.g.johnston@gmail.com> wrote:
> >
> > On Wednesday, April 15, 2026, Amit Kapila <amit.kapila16@gmail.com> wro=
te:
> >>
> >>
> >> 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=E2=80=99d 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:
> >
> > =E2=80=9CThe new cluster is only permitted to have temporary logical sl=
ots, i.e., ones where pg_replication_slots.temporary is true.=E2=80=9D
> >
>
> 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

--00000000000050c232064fa14671
Content-Type: application/octet-stream;
	name="v2-0001-Improve-the-wording-of-logical-slot-prerequisites.patch"
Content-Disposition: attachment;
	filename="v2-0001-Improve-the-wording-of-logical-slot-prerequisites.patch"
Content-Transfer-Encoding: base64
Content-ID: <f_mo2h1a4w0>
X-Attachment-Id: f_mo2h1a4w0

RnJvbSBkODkxODAyZmQ4ZmFlOWRhYTU0NGM1MjI5YmYxNjc0ZjEzMWU1NDIxIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBWaWduZXNoIEMgPHZpZ25lc2gyMUBnbWFpbC5jb20+CkRhdGU6
IFRodSwgMTYgQXByIDIwMjYgMTI6MzY6MzQgKzA1MzAKU3ViamVjdDogW1BBVENIIHYyXSBJbXBy
b3ZlIHRoZSB3b3JkaW5nIG9mIGxvZ2ljYWwgc2xvdCBwcmVyZXF1aXNpdGVzIHRvIG1ha2UKIHRo
ZSBjb25kaXRpb25zIGVhc2llciB0byB1bmRlcnN0YW5kLgoKUmVwbGFjZSB0aGUgcHJldmlvdXMg
bmVnYXRpdmUgcGhyYXNpbmcgc3VjaCBhcyAidGhlcmUgYXJlIG5vIHNsb3RzCndob3NlIC4uLiBp
cyBub3QgdHJ1ZSIgd2l0aCBhIGRpcmVjdCBleHByZXNzaW9uIHRoYXQgYWxsIHNsb3RzIG11c3QK
aGF2ZSBjb25mbGljdGluZyA9IGZhbHNlLgoKU2ltaWxhcmx5LCByZXdvcmQgdGhlIHJlcXVpcmVt
ZW50IG9uIHRoZSBuZXcgY2x1c3RlciB0byBzdGF0ZSB0aGF0IGl0Cm11c3Qgbm90IGhhdmUgYW55
IHBlcm1hbmVudCBsb2dpY2FsIHNsb3RzLCBjbGFyaWZ5aW5nIHRoYXQgYW55IGV4aXN0aW5nCmxv
Z2ljYWwgc2xvdHMgbXVzdCBoYXZlIHRlbXBvcmFyeSBzZXQgdG8gdHJ1ZS4KClRoZXNlIGNoYW5n
ZXMgaW1wcm92ZSByZWFkYWJpbGl0eSB3aXRob3V0IGFsdGVyaW5nIHRoZSBtZWFuaW5nLgoKQXV0
aG9yOiBWaWduZXNoIEMgPHZpZ25lc2gyMUBnbWFpbC5jb20+ClJldmlld2VkLWJ5OiBBbWl0IEth
cGlsYSA8YW1pdC5rYXBpbGExNkBnbWFpbC5jb20+ClJldmlld2VkLWJ5OiBEYXZpZCBHLiBKb2hu
c3RvbiA8ZGF2aWQuZy5qb2huc3RvbkBnbWFpbC5jb20+CkRpc2N1c3Npb246IGh0dHBzOi8vd3d3
LnBvc3RncmVzcWwub3JnL21lc3NhZ2UtaWQvZmxhdC8xNzc2MDkyNzg3MzcuNDAzMDU5LjE0MTc0
Mjc1MDEzMDkwNDcxOTQ3JTQwd3JpZ2xleXMucG9zdGdyZXNxbC5vcmcKLS0tCiBkb2Mvc3JjL3Nn
bWwvbG9naWNhbC1yZXBsaWNhdGlvbi5zZ21sIHwgMTEgKysrKystLS0tLS0KIDEgZmlsZSBjaGFu
Z2VkLCA1IGluc2VydGlvbnMoKyksIDYgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvZG9jL3Ny
Yy9zZ21sL2xvZ2ljYWwtcmVwbGljYXRpb24uc2dtbCBiL2RvYy9zcmMvc2dtbC9sb2dpY2FsLXJl
cGxpY2F0aW9uLnNnbWwKaW5kZXggMjNiMjY4MjczYjkuLjU5OGUyM2FkNGY1IDEwMDY0NAotLS0g
YS9kb2Mvc3JjL3NnbWwvbG9naWNhbC1yZXBsaWNhdGlvbi5zZ21sCisrKyBiL2RvYy9zcmMvc2dt
bC9sb2dpY2FsLXJlcGxpY2F0aW9uLnNnbWwKQEAgLTI3ODUsMTggKzI3ODUsMTcgQEAgQ09OVEVY
VDogIHByb2Nlc3NpbmcgcmVtb3RlIGRhdGEgZm9yIHJlcGxpY2F0aW9uIG9yaWdpbiAicGdfMTYz
OTUiIGR1cmluZyAiSU5TRVIKICAgICA8L2xpc3RpdGVtPgogICAgIDxsaXN0aXRlbT4KICAgICAg
PHBhcmE+Ci0gICAgICBBbGwgc2xvdHMgb24gdGhlIG9sZCBjbHVzdGVyIG11c3QgYmUgdXNhYmxl
LCBpLmUuLCB0aGVyZSBhcmUgbm8gc2xvdHMKLSAgICAgIHdob3NlCisgICAgICBBbGwgc2xvdHMg
b24gdGhlIG9sZCBjbHVzdGVyIG11c3QgYmUgdXNhYmxlLCBpLmUuLCB0aGVpcgogICAgICAgPGxp
bmsgbGlua2VuZD0idmlldy1wZy1yZXBsaWNhdGlvbi1zbG90cyI+cGdfcmVwbGljYXRpb25fc2xv
dHM8L2xpbms+LjxzdHJ1Y3RmaWVsZD5jb25mbGljdGluZzwvc3RydWN0ZmllbGQ+Ci0gICAgICBp
cyBub3QgPGxpdGVyYWw+dHJ1ZTwvbGl0ZXJhbD4uCisgICAgICBpcyA8bGl0ZXJhbD5mYWxzZTwv
bGl0ZXJhbD4uCiAgICAgIDwvcGFyYT4KICAgICA8L2xpc3RpdGVtPgogICAgIDxsaXN0aXRlbT4K
ICAgICAgPHBhcmE+Ci0gICAgICBUaGUgbmV3IGNsdXN0ZXIgbXVzdCBub3QgaGF2ZSBwZXJtYW5l
bnQgbG9naWNhbCBzbG90cywgaS5lLiwKLSAgICAgIHRoZXJlIG11c3QgYmUgbm8gc2xvdHMgd2hl
cmUKKyAgICAgIFRoZSBuZXcgY2x1c3RlciBtdXN0IG5vdCBoYXZlIGFueSBwZXJtYW5lbnQgbG9n
aWNhbCBzbG90czsgaS5lLiwgYW55CisgICAgICBleGlzdGluZyBsb2dpY2FsIHNsb3RzIG11c3Qg
aGF2ZQogICAgICAgPGxpbmsgbGlua2VuZD0idmlldy1wZy1yZXBsaWNhdGlvbi1zbG90cyI+cGdf
cmVwbGljYXRpb25fc2xvdHM8L2xpbms+LjxzdHJ1Y3RmaWVsZD50ZW1wb3Jhcnk8L3N0cnVjdGZp
ZWxkPgotICAgICAgaXMgPGxpdGVyYWw+ZmFsc2U8L2xpdGVyYWw+LgorICAgICAgc2V0IHRvIDxs
aXRlcmFsPnRydWU8L2xpdGVyYWw+LgogICAgICA8L3BhcmE+CiAgICAgPC9saXN0aXRlbT4KICAg
IDwvaXRlbWl6ZWRsaXN0PgotLSAKMi40My4wCgo=
--00000000000050c232064fa14671--