Date: Sun, 23 May 2021 17:38:31 +0900
From: Michael Paquier <michael@paquier.xyz>
To: Amit Langote <amitlangote09@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Amit Kapila <amit.kapila16@gmail.com>,
	PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: Subscription tests fail under CLOBBER_CACHE_ALWAYS
Message-ID: <YKoUh18SDF2eLRX7@paquier.xyz>
References: <YKSciAJEPQ7wmNDt@paquier.xyz>
 <CA+HiwqEW7kVF+VKdHT7TnzQGab9-dP9tLyedVvs5nQ7yJ_BP6g@mail.gmail.com>
 <3508426.1621455835@sss.pgh.pa.us>
 <CA+HiwqF6HXDTnkcb07HV+ErzcYpAL_0_MwviNwGi+VkzbYdkEQ@mail.gmail.com>
 <162691.1621630908@sss.pgh.pa.us>
 <CA+HiwqFnbV7W7j4S3s8pKaCe58stWMFzTsvUgC7SP2WKC-aSRQ@mail.gmail.com>
 <228806.1621689537@sss.pgh.pa.us>
 <264714.1621697563@sss.pgh.pa.us>
 <415326.1621733300@sss.pgh.pa.us>
 <CA+HiwqGWt8AWe8PKYhPMH7AmSq6SH4XAPn7AwfcDUrED7Ju-HA@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="S6D7YYa3/61vA1cZ"
Content-Disposition: inline
In-Reply-To: 
 <CA+HiwqGWt8AWe8PKYhPMH7AmSq6SH4XAPn7AwfcDUrED7Ju-HA@mail.gmail.com>
Archived-At: 
 <https://www.postgresql.org/message-id/YKoUh18SDF2eLRX7%40paquier.xyz>
Precedence: bulk


--S6D7YYa3/61vA1cZ
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, May 23, 2021 at 02:05:59PM +0900, Amit Langote wrote:
> On Sun, May 23, 2021 at 10:28 AM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> I wrote:
>> > ... wrong.  Running v13 branch tip under CLOBBER_CACHE_ALWAYS provokes
>> > a core dump in 013_partition.pl, so 1375422c is not to blame.  Now
>> > I'm wondering how far back there's a live issue.
>>
>> Oh, of course, it's directly the fault of the patch that added support
>> for partitioned target tables.
>=20
> Yeah, the problem seems to affect only partition child tables, so yeah
> this problem started with f1ac27bfda6.

Yep.

>> I concluded that a verbatim backpatch wasn't too suitable because
>> a04daa97a had changed a lot of the APIs here.  So I left the APIs
>> for the xxx_internal() functions alone.  Otherwise the patch
>> pretty much works as-is in v13.

Thanks for the backpatch of the partition tests via d18ee6f.
--
Michael

--S6D7YYa3/61vA1cZ
Content-Type: application/pgp-signature; name="signature.asc"

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

iQIzBAABCgAdFiEEG72nH6vTowiyblFKnvQgOdbyQH0FAmCqFIcACgkQnvQgOdby
QH0dNg/+NQc8+75ZGMfNhWT8hghXNH4bF50XPpOV7iGSM3gBTYDMbU3Ap4gupBJm
/UdXSktO/L7zZ7vj1SZMq9UOjOhH/AGi/gB9mycoYM2snirlU0sT7ys0Q9zfRV9Q
4tvmUrZD6A+OcGeZ2NTKBU+0Nm+1W9eIRSb2ExaePgaWJ9OV9qtJUn9V9oEvBE5a
xGX5k1bW/G7Da9uD9LuPNmN+0WlPGJCSW8zTZUNO3ZsZXprYEm5BDQEdSNzBg/Ig
gFalHcK8Vopo0Z0Iqbd4lbKXQJzWH+7hKvNmLeASEHXWRrD7zaJvBwJLatnM7+2F
LusOBgB+BolE/Y99PtFJLH43oZe50gExKpbkZiyvu3R+6vSe8KNLi/3co9THMPSj
lJl+U1n2ZZ4PMuWbRbPuruYSjC+/fZNyIeRzvK9ybjZlVl4R8Vvs/Q8ArST1Iqd1
xDHK3EhiaSCHwlS7dyrOur86jsCfMMXbopkwkvP2R3WAbYNk7Y9Zz1ygCY8cCMCr
aWhx6nWZD8EcK1v4Ax/XF3gt7t+dwaRj8JWpy/jOFYKR3cw9fqjSONsNFrao2Itx
hmWExQIuWwE+VHtOpgsblEm2/62S1E+uJ/GDrNKBfoGWt5WmeFv9wwkF6gX2BMq0
LpYt8ejtL9k+HOQfn0R9YkPI1P9poMpIKwuUGrtPiaVOBqZgALE=
=sGhd
-----END PGP SIGNATURE-----

--S6D7YYa3/61vA1cZ--