pgsql: Have read_stream_reset reset pending_read_nblocks.

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Have read_stream_reset reset pending_read_nblocks.
Date: 2026-09-15 18:37:28
Message-ID: E1x6Y24-00000000Tek-1lWO@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

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 <tomas(at)vondra(dot)me>
Discussion: https://postgr.es/m/CAH2-WzkKY8b42sZSPWJceT5Fn_THk3ZgcaM8ch3Hzw+XvhP0zQ@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(+)

Browse pgsql-committers by date

  From Date Subject
Next Message Masahiko Sawada 2026-09-15 22:07:43 pgsql: Use hex_decode_safe() to speed up UUID input.
Previous Message Álvaro Herrera 2026-09-15 16:28:25 pgsql: Fail REPACK in presence of invalid indexes, take 2