Fix Parallel TID Range Scan rescans with a different TID range
authorDavid Rowley <drowley@postgresql.org>
Fri, 16 Oct 2026 03:49:16 +0000 (16:49 +1300)
committerDavid Rowley <drowley@postgresql.org>
Fri, 16 Oct 2026 03:49:16 +0000 (16:49 +1300)
commitba51d62be465c420b0fbe3b0efd0ccd488e79922
treef0805f63f0868139d7666950c4275fecf281ae09
parent856a6594d86e4a7c0e9490253912af3f6159802f
Fix Parallel TID Range Scan rescans with a different TID range

0ca3b1697 added support for parallel TID Range Scans.  The range of
blocks to scan is stored in the shared phs_startblock and phs_numblock
fields by the first participant to start the scan, but
table_block_parallelscan_reinitialize() didn't reset these, so every
rescan of the Gather reused the block range of the first execution.
This caused rows to be silently missed when a rescan used a different
TID range, e.g:

SELECT r.lo, (SELECT count(*) FROM t WHERE ctid >= (SELECT r.lo))
FROM (VALUES ('(30,0)'::tid), ('(10,0)'), ('(0,0)')) r(lo);

Here we fix this by resetting both fields in
table_block_parallelscan_reinitialize().  As a side effect, a rescanned
Parallel Seq Scan using synchronized scans now gets a new start
location, which is what a new scan would do anyway.

The existing rescan test didn't exercise this, so add one that does.

This was introduced in 0ca3b1697, so backpatch to v19.

Reported-by: Zsolt Parragi <zsolt.parragi@percona.com>
Author: Zsolt Parragi <zsolt.parragi@percona.com>
Author: David Rowley <dgrowleyml@gmail.com>
Reviewed-by: David Rowley <dgrowleyml@gmail.com>
Reviewed-by: shihao zhong <zhong950419@gmail.com>
Discussion: https://postgr.es/m/CAN4CZFOoyCQE483YVGg07_Vr2MjFCTiCLUHUAFCwKqJAE=PN6w@mail.gmail.com
Backpatch-through: 19
src/backend/access/table/tableam.c
src/test/regress/expected/tidrangescan.out
src/test/regress/sql/tidrangescan.sql