Have read_stream_reset reset pending_read_nblocks.

Otherwise, a read of blocks the stream is about to throw away can be
started and then waited for.  read_stream_reset drains the stream by
calling read_stream_next_buffer, whose look-ahead treats the zeroed
distance as end of stream, leading to it spuriously performing the
pending read.  This caused a pathological regression for a nested loop
antijoin query when run with the upcoming index prefetching patch series
applied.

This bug affects all stable branches with the read stream, which was
added to Postgres 17 by commit b5a9b18cd0b.  No backpatch for now,
though, since without index prefetching the added overhead for realistic
query plans is negligible.

Author: Tomas Vondra <[email protected]>
Discussion: 
https://postgr.es/m/cah2-wzkky8b42szspwjcet5fn_thk3zgcam8ch3hzw+xvhp...@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/862092932c9479b79732f3b441da05453ea5e06d

Modified Files
--------------
src/backend/storage/aio/read_stream.c | 3 +++
1 file changed, 3 insertions(+)

Reply via email to