To: Alvaro Herrera <alvherre@2ndquadrant.com>,
 Dave Cramer <davecramer@postgres.rocks>
Cc: Andres Freund <andres@anarazel.de>, Michael Paquier
 <michael@paquier.xyz>, Fujii Masao <masao.fujii@oss.nttdata.com>,
 Masahiko Sawada <masahiko.sawada@2ndquadrant.com>,
 Kyotaro Horiguchi <horikyota.ntt@gmail.com>,
 Vladimir Sitnikov <sitnikov.vladimir@gmail.com>,
 PostgreSQL-development <pgsql-hackers@postgresql.org>,
 Peter Geoghegan <pg@bowt.ie>
References: <20200605155148.GA19795@alvherre.pgsql>
From: "Jonathan S. Katz" <jkatz@postgresql.org>
Autocrypt: addr=jkatz@postgresql.org; keydata=
 mQINBFtShwABEAC7PNHDUOTYuifpUCk23KqfxdQQkn4nkoxOXRK0+rAj36FiwqGB4TJFuOVZ
 sDFAEiSlC8Jt4y5Cs7B5tetT8JNd2cs6zp/udMJJDz9d65O9PDpdlMgAmIiTzpLlSdx8FG56
 DTksaDv1d8j3cTJPSE4/fWSxqzA7o3Y9UuL7atZPrzfImgpRKs0of7elIHwOa8GucjyhYqcR
 h60wFBJc2KXqQdDYRTZy43DSnY/0VNc0omiH355fustvpm+m5HjD3w7qZyfN3fpKJpnX1LCF
 f3MnPHaDGITIYGRCBXvf0UqUtD6OEVWPv2C2gyqWMIpWmZTOgDufltKyIByKBoS9x0PlFkij
 04X3KODCngt+N8Ssc9OICc6QSxhjoP48PYPdmiTmkrGuf0LX084wj1xeo1NX7XxZK39F6dTJ
 DhsIiW0sNS0xMxQHLHG9VLbPjx3SANQBh6BuryPz5ZupW9/TIDmkvprtU/oXfKgtfYm3fxmk
 EctxbWrEPsFTFPyuMqQu6l+xyQv0s1VLZfjNWaua6H1/gGoIt6kRnn5qMXDVVpijuWkHbv7G
 ngaQMd258UrrOEHnnjzhQ7jxMWV9D+emxbAtlIxnYvCWlV4IwAQhEHfvudqYaIY3hNWrvQ6H
 GB2KXoTZYN9g5djm14/5nj1IU5zOcovkjJnKhoo9iStnpFF2cwARAQABtCdKb25hdGhhbiBT
 LiBLYXR6IDxqa2F0ekBwb3N0Z3Jlc3FsLm9yZz6JAlQEEwEIAD4CGwMFCQeGH4AFCwkIBwIG
 FQoJCAsCBBYCAwECHgECF4AWIQT6hLaVryv3miBkP/HxBJxynxxlJwUCW1KJJQAKCRDxBJxy
 nxxlJwjrD/kBgqsW4QpNpTFw7ifRokZV08CCX4huPBJQ91rrv+UEWlEcotFBHVkYyHnpzARl
 tcZxhJ9CbFxjniH9cOTty5T/O1yolbOHtZSW8Z8aWV6BVEbjMb+BFxSSLm7RnvJdzQbGCZq2
 ZZvfVpB6z3EHYph4KDdVKvMFjoLskxmdS1DE0tE3zTxvoQsi24Q+HOS07kUjs6fsu/WICMfz
 mgO++AWG9Y0CvN0mm4TkujESzyKM9E5irD+leEMIcddl51Aa2c/VMfBXQbRmpHIgUFTmuHQD
 CnQih+9i3OJAksDg66SP8a7yiXv5mwvyDi1EfTGVKYR2j+pwyjwnC3oIbvDMmB3uTn2JIjnT
 iZKPVtAcAylXjubFltihQgNyuShdP4W+kBwZizhUFqUVL8Anx+KoytYmJPfMRFLGuK4obXKq
 a2ZS3k9KB+H+isOx2nFJOsc7V360Zp1DVaNmuiK10TT6QndShSPaqkJqFtCb6r92rZ9sZM/L
 3vtCI4Rrl3Pt1MgtENXupS8gZpJnAYS0j5A1PAZ09r6ANoaeMHspF+5J5fOHeEvqphXr36mm
 a83Vl1t4orPb0+QmmijmlpseDU63M88Aw5p3c4qj7t8Qr2EZ5zrn7/sFn5wOfbs8Nymxafif
 QCnlV2vg9p0m7vSk/yLJ4PFZvs52FgqAGRCdRn0s2EC99bkCDQRbUocAARAAv8ho/toQ9DG3
 j4f9h9n1aRHr2FlviN2Utpy6L8+dfDggO0geilmkGQOolZ2E60gGfye/kUtF9W3NByO4hxDR
 9u6qbOXcdqnuA+cc68EfqlWFJrVtYFxt0h4ElWYOYnIezKthriWch/FY70FGrxs3z8UHOHq5
 0wBW433eTvZm90WixBiXEt2v1DgW4Vr3ymfO7Aap/IYyPuE4JzgudAuAl0HKPyNEHWHG1dAb
 jX1RiCw9gknIDWQOF0B4UAaJctWGVcnZ3A2ULwNGMa1P9ZJlBWf1vcj01aiHMU0yQ7JjJiSp
 vfm9eM0uSLwRdDrJjyl5ZZqVumjdv2SMNQ8GvYRbEMys3GGDSt9zXgfCSUnPnJfYxjzBHRI6
 x44Wfsx8S6hWxepOogCJJ/g67Bk9mY8YV4klWIXDJVOL5jnBC09DbsZG81JaE2QxB8Y7W36Z
 Mroi9XMxg3s805hQAQUvdG/poU8hN8BWdrnTm/+4eQQp7gDY1ePDmGM6bJC+OHOSnFtR/f+7
 0zpKJ10cc7cBygGnl1yR3KjhFyAWUFvP4ZGziKCcpMwXZfe9PGuyA/YOubMphxIn3YsK2wrd
 faKZYX2GMZCZhMMcvx9IpQrxIJgU+VlwXu/O+Lk10VIPcxPJJwmpdI6HzcS8ZgG6IMcC444X
 XTuLaP8j2mgcMvYak3ScCykAEQEAAYkCPAQYAQgAJhYhBPqEtpWvK/eaIGQ/8fEEnHKfHGUn
 BQJbUocAAhsMBQkHhh+AAAoJEPEEnHKfHGUnReMP/RA2UhGQj+G4uBshkRLjpRysabdPqgQB
 dEBk6wYbio88Wg/2/hgY7UzmDDEwX3sZfQDcrI6+vIobI8uqstZID+WgAAa1JLfChMyVQnSy
 0zfWMOABXscc5tGuvFRZvJklTissMFjXUwaREEKp4ZikTvJ/62MCjSdtrUhnPLvoTsHTKRKD
 ichE+b5A54alwsubTijw12O6N22r5IjZiiSZV0u7dsShyKw+7wCSax9fuBoE43NMYf+dnjMK
 nerAQYUcZWYMnk+EC8RaqYAxv6XZ2tKx1AkGGktwkQIBwrz9IlDSvJ3LWJ2UIIuLRTdngNgg
 GIL4zzuUa2F56FqskQIuYMaNETk6LYfalBDQ6TVLAcgCPQxp4k4i/PRsZ3lZ2ZhRHRYciOvm
 kp+I6EfHwllQpYrWs1thluBGqlJSVJgKl0IOFvKLsQ6KKfqzAwh6FxrO5qajp4viNIgtWoFw
 O5Bp0jgFTbH2OrMWIRfUdUCH1Djbuo65svhNj7FNsQVYzHDI+Nd1I/LOBoPc3UorRMF2M4JV
 kUR7skOHWUbPTSNUr7qc14NSMY3PKjGeVGAsBVHBPvmRx/Ss2tW/TpJWpxg4pmquFQSXuaYh
 Yf4FN8Sxy320pcr+FqN8AhZkYJcNY82OTtR5VEKRC/mYcyq61qXIKqngydn79bJjGxHipxyF FmEC
Subject: Re: SIGSEGV from START_REPLICATION 0/XXXXXXX in XLogSendPhysical ()
 at walsender.c:2762
Message-ID: <69edce5b-c5a4-6b1a-fe2f-9e53d1e0d19f@postgresql.org>
Date: Sun, 21 Jun 2020 13:45:36 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:68.0)
 Gecko/20100101 Thunderbird/68.9.0
MIME-Version: 1.0
In-Reply-To: <20200605155148.GA19795@alvherre.pgsql>
Content-Type: multipart/signed; micalg=pgp-sha256;
 protocol="application/pgp-signature";
 boundary="DIjs1RqPBMDLtn7yTe9J64Q9gdMdNqVdF"
Precedence: bulk

This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--DIjs1RqPBMDLtn7yTe9J64Q9gdMdNqVdF
Content-Type: multipart/mixed; boundary="65xzrE4fbLRWpossEbPu0yJ2HQiHIb50x";
 protected-headers="v1"
From: "Jonathan S. Katz" <jkatz@postgresql.org>
To: Alvaro Herrera <alvherre@2ndquadrant.com>,
 Dave Cramer <davecramer@postgres.rocks>
Cc: Andres Freund <andres@anarazel.de>, Michael Paquier
 <michael@paquier.xyz>, Fujii Masao <masao.fujii@oss.nttdata.com>,
 Masahiko Sawada <masahiko.sawada@2ndquadrant.com>,
 Kyotaro Horiguchi <horikyota.ntt@gmail.com>,
 Vladimir Sitnikov <sitnikov.vladimir@gmail.com>,
 PostgreSQL-development <pgsql-hackers@postgresql.org>,
 Peter Geoghegan <pg@bowt.ie>
Message-ID: <69edce5b-c5a4-6b1a-fe2f-9e53d1e0d19f@postgresql.org>
Subject: Re: SIGSEGV from START_REPLICATION 0/XXXXXXX in XLogSendPhysical ()
 at walsender.c:2762
References: <20200605155148.GA19795@alvherre.pgsql>
In-Reply-To: <20200605155148.GA19795@alvherre.pgsql>

--65xzrE4fbLRWpossEbPu0yJ2HQiHIb50x
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: quoted-printable

Hi,

On 6/5/20 11:51 AM, Alvaro Herrera wrote:
> On 2020-Jun-05, Dave Cramer wrote:
>=20
>> On Thu, 4 Jun 2020 at 19:46, Alvaro Herrera <alvherre@2ndquadrant.com>=

>> wrote:
>=20
>>> Ouch ... so they made IDENT in the replication grammar be a trigger t=
o
>>> enter the regular grammar.  Crazy.  No way to put those worms back in=

>>> the tin now, I guess.
>>
>> Is that documented ?
>=20
> I don't think it is.
>=20
>>> It is still my opinion that we should prohibit a logical replication
>>> connection from being used to do physical replication.  Horiguchi-san=
,
>>> Sawada-san and Masao-san are all of the same opinion.  Dave Cramer (o=
f
>>> the JDBC team) is not opposed to the change -- he says they're just
>>> using it because they didn't realize they should be doing differently=
=2E
>>
>> I think my exact words were
>>
>> "I don't see this is a valid reason to keep doing something. If it is
>> broken then fix it.
>> Clients can deal with the change."
>>
>> in response to:
>>
>>> Well, I don't really think that we can just break a behavior that
>>> exists since 9.4 as you could break applications relying on the
>>> existing behavior, and that's also the point of Vladimir upthread.
>>
>> Which is different than not being opposed to the change. I don't see t=
his
>> as broken, and it's quite possible that some of our users are using
>> it.
>=20
> Apologies for misinterpreting.
>=20
>> It certainly needs to be documented
>=20
> I'd rather not.

The PG13 RMT had a discussion about this thread, and while the initial
crash has been fixed, we decided to re-open the Open Item around whether
we should allow physical replication to be initiated in a logical
replication session.

We anticipate a resolution for PG13, whether it is explicitly
disallowing physical replication from occurring on a logical replication
slot, maintaining the status quo, or something else such that there is
consensus on the approach.

Thanks,

Jonathan


--65xzrE4fbLRWpossEbPu0yJ2HQiHIb50x--

--DIjs1RqPBMDLtn7yTe9J64Q9gdMdNqVdF
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEE+oS2la8r95ogZD/x8QSccp8cZScFAl7vnMAACgkQ8QSccp8c
ZSdOtw/+MM+v25zMmRl3vq5zMee6t7/WK3l7dFgMhLKaFC0ZNqha61bXda58fsjl
jx8dHTC2oV6c1tg5GOhzJQ+px6y0XtaT6PP1sUEqiBKAWkFPYquQKQFpCDwZmEGJ
4UzVfYL1T8kQbDfId2zG7KhCiOKZKWK2BA3Xd957mB8Rka3ttExGI5ljAcX0SBTy
PvrOdI+B26VUSn/Y27CrSrUKJUIOFmbTlQOv+1w8ueMSNTYoirTBitPbbLEJugSg
ospyFd/2ovQos0TfsQ5rQHWKXG2Jp7ebgNp26nwdcA4ofHtdObvFwVLqUfs+Q7Ca
Q119CCPjQVJ9wEjUJYpBqEfB8h4Yyw26u1qMrIKPXsLw0DVoSLURy7iKten/NIpE
BGkChbBfqPDEF6SKcQnefJlzY0Ah1gjon+mEZ4Pu7rV2Ow8yQU+fHJTifqh6czmI
a7S3ChwE1F+1yefRhpt6W/zsb9+3vYR6QHYhs2Z8DwSh8cX3GH0aY0p/19SXYPpF
WJvGyRt07O1DdYgb+nurmnYfyGfV72oB6j/E03zHdk5ml1XfHvo7RM3IGM1PFUyC
2t8D9IWXBKeVoH1UtTPm7wHpi2dlshvNSjoroRCXW88MFIjWsAPPL5bLIOcr76G0
Xh+a0ntl9bJwI4/JukMwraQNSYjUvh1IqLrgvN5T2vdGZDnl4X0=
=h6yY
-----END PGP SIGNATURE-----

--DIjs1RqPBMDLtn7yTe9J64Q9gdMdNqVdF--