MIME-Version: 1.0
References: <zljergweqti7x67lg5ije2rzjusie37nslsnkjkkby4laqqbfw@3p3zu522yykv>
 <CA+hUKGKwzPJNcxCFUswe0K=0e7rG79dvqt9o17-E3soxLPxF+Q@mail.gmail.com>
 <CAAKRu_atdnPCCy=kfxqWT62Ckaiz3G5t=S97tW24CuL3i3fFfQ@mail.gmail.com>
 <CAN55FZ17ScU--nGM8cjjPtwPMOMonKZ9vMehPcc2sOQNLVskvA@mail.gmail.com>
 <CAAKRu_ZM1epxTdt2=4-g4ff6UC09zne+0xg_gNv3d7LEcxEvNA@mail.gmail.com>
In-Reply-To: 
 <CAAKRu_ZM1epxTdt2=4-g4ff6UC09zne+0xg_gNv3d7LEcxEvNA@mail.gmail.com>
From: Melanie Plageman <melanieplageman@gmail.com>
Date: Tue, 3 Mar 2026 15:07:17 -0500
Message-ID: 
 <CAAKRu_bAfvo84NKPRT--wUdV5YtHaYqo1=-QiKkiaDSjDPB9hg@mail.gmail.com>
Subject: Re: Don't synchronously wait for already-in-progress IO in read
 stream
To: Nazir Bilal Yavuz <byavuz81@gmail.com>
Cc: Thomas Munro <thomas.munro@gmail.com>, Andres Freund <andres@anarazel.de>,
	pgsql-hackers@postgresql.org, Peter Geoghegan <pg@bowt.ie>,
 Tomas Vondra <tv@fuzzy.cz>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAAKRu_bAfvo84NKPRT--wUdV5YtHaYqo1%3D-QiKkiaDSjDPB9hg%40mail.gmail.com>
Precedence: bulk

On Tue, Mar 3, 2026 at 2:47=E2=80=AFPM Melanie Plageman
<melanieplageman@gmail.com> wrote:
>
> Some ideas I had that I didn't include in v4-0003 because its Andres
> patch and is subjective:

I was just looking at another patch and realized test_read_stream.c
exists. I wonder if any of the code this patch set adds to test_aio.c
should be there? On the one hand the foreign IO test is testing AIO
behavior and not really read stream behavior even though it invokes
the read stream. So maybe it doesn't really belong in
0004_read_stream.pl?  The repeated blocks test is more of a read
stream test. Anyway, just a thought I had.

- Melanie