Feedback-ID: i0fe9450f:Fastmail
Date: Thu, 11 Jun 2026 18:53:19 +0900
From: Michael Paquier <michael@paquier.xyz>
To: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Cc: Xuneng Zhou <xunengzhou@gmail.com>,
	"Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>,
	Alexander Lakhin <exclusion@gmail.com>,
	pgsql-hackers <pgsql-hackers@postgresql.org>
Subject: Re: t/035_standby_logical_decoding.pl might fail on attempt to read
 wrong timeline
Message-ID: <aiqFjzGHpnYFP-Gm@paquier.xyz>
References: 
 <OS9PR01MB1214908BA67A7811BD6281208F51C2@OS9PR01MB12149.jpnprd01.prod.outlook.com>
 <CABPTF7WmLKOJcSPod7zT2hynUFQcUs0VoQyR-p=XFsafvmGm7g@mail.gmail.com>
 <CABPTF7U_B9pdC563fFONQLX_FXCZtxZgM+jN9snzMVg9b9MRfg@mail.gmail.com>
 <aibQB2f9h3nYeg/2@bdtpg>
 <CABPTF7XGDgnB97fBUQb+Tm=5NnAzS1iKTWcJTePpGinL1ZAxWQ@mail.gmail.com>
 <CABPTF7VovNWnH=dG_A=e6xkMYt11ubbj935Ymfky55cXcWcrrA@mail.gmail.com>
 <aifyiJG3uiJzCYTB@bdtpg>
 <CABPTF7XAb7ExE-7qSsKgSw3K0hfsNPzXuupQ1aJ8zXOgZ4tPNw@mail.gmail.com>
 <aimeoNYE93VLiQHt@bdtpg>
 <aioMFYcZhLB6urla@paquier.xyz>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="8QsCYnoxh4/STvAp"
Content-Disposition: inline
In-Reply-To: <aioMFYcZhLB6urla@paquier.xyz>
Archived-At: 
 <https://www.postgresql.org/message-id/aiqFjzGHpnYFP-Gm%40paquier.xyz>
Precedence: bulk


--8QsCYnoxh4/STvAp
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Thu, Jun 11, 2026 at 10:15:01AM +0900, Michael Paquier wrote:
> The backpatch of PG16 is straight-forward and adding
> GetWALInsertionTimeLineIfSet() down there does not look like an issue.
> Not having any tests in v16 feels sad, but that's life.  It does not
> prevent addressing the availability issue on this branch.

Backpatch of GetWALInsertionTimeLineIfSet() done in v16.

> I'll go take it up from here.

I have spent some time on the first part of this patch series,
focusing on the noisiest path for logical replication, and applied it
down to v17.  I have been able to reproduce the problem using
Alexander's sleep trick, and let 035 run with the same trick and the
patch applied for a few hundred runs.

Note that REL_17_STABLE has currently 4 failures in the buildfarm, due
to me messing up with the existence of injection_points.  I'll
probably just remove the test on this branch, 105b2cb33617 adding the
requirement on the module only in v18~.
--
Michael

--8QsCYnoxh4/STvAp
Content-Type: application/pgp-signature; name=signature.asc

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

iQIzBAEBCgAdFiEEG72nH6vTowiyblFKnvQgOdbyQH0FAmoqhY8ACgkQnvQgOdby
QH314A//QHZOfekegrTBOWGvrktrzjr0P+VDZGrd2fEC1j2xvAyRw01tz9A6L3Y1
ieNm7XTXJIdNai4MTzYyy5fN8Q1NLF93hPWQfQeG0utBKKsyzFGIjA+3aczscwNO
5k8ZZXyriuFo9ic6HCGo3for/0OV81LX5uH0TTdN1ViH136de/YIUEJSdHgAD3Ci
EYVQ3S8OU8V2FoDGJPCHY1yIYe/7+Be7Nuquw3oVkLXbPA6wunZuGzEIWj0ReY+e
wRycMEP2X90s3/6ktgtkV/i1xWpKvhdk+0GETurvHjkdAa1D5J78TfnazKqXiaXn
EEi7XMNkZXXtUHbTWWT5rS3EAHr99uBGY85/ffyzcrWzpvPWMPyNMAkZe4aClW+3
Vw7bNbnYjGo0fRE22/9JLFtjc+pyUZtOyzX4eW1/Lih6/eUI18itPsttDpG+oq6g
TmVVRttFyDfEaD1blQrGUunXLBm+5JTYgLRQJ7rILCHqIuSYiUSsJqtRJhFs9Wlk
8/P1ln1lU1rC2+HedaRFp2Cl7oK6e3MMuNQ7Wfh8J/0lv0pARVsVg7tgYhQFioZD
xHmUJHblXg3uTdzbP3AHNIefgRT0DaPl2W80YFP7o28pZVFD4Qgexr73sIEj0+1K
v1pYCMXPM9EzJbwF2sAD8YKZbUzm9eiW8l8c8jv/2wnxT1EEWTM=
=1SNc
-----END PGP SIGNATURE-----

--8QsCYnoxh4/STvAp--