Commit 8ce29bb4 authored by Amit Kapila's avatar Amit Kapila

Fix the buffer release order for parallel index scans.

During parallel index scans, if the current page to be read is deleted, we
skip it and try to get the next page for a scan without releasing the buffer
lock on the current page.  To get the next page, sometimes it needs to wait
for another process to complete its scan and advance it to the next page.
Now, it is quite possible that the master backend has errored out before
advancing the scan and issued a termination signal for all workers.  The
workers failed to notice the termination request during wait because the
interrupts are held due to buffer lock on the previous page.  This lead to
all workers being stuck.

The fix is to release the buffer lock on current page before trying to get
the next page.  We are already doing same in backward scans, but missed
it for forward scans.

Reported-by: Victor Yegorov
Bug: 15290
Diagnosed-by: Thomas Munro and Amit Kapila
Author: Amit Kapila
Reviewed-by: Thomas Munro
Tested-By: Thomas Munro and Victor Yegorov
Backpatch-through: 10 where parallel index scans were introduced
Discussion: https://postgr.es/m/153228422922.1395.1746424054206154747@wrigleys.postgresql.org
parent 39d51fe8
...@@ -1497,19 +1497,21 @@ _bt_readnextpage(IndexScanDesc scan, BlockNumber blkno, ScanDirection dir) ...@@ -1497,19 +1497,21 @@ _bt_readnextpage(IndexScanDesc scan, BlockNumber blkno, ScanDirection dir)
/* nope, keep going */ /* nope, keep going */
if (scan->parallel_scan != NULL) if (scan->parallel_scan != NULL)
{ {
_bt_relbuf(rel, so->currPos.buf);
status = _bt_parallel_seize(scan, &blkno); status = _bt_parallel_seize(scan, &blkno);
if (!status) if (!status)
{ {
_bt_relbuf(rel, so->currPos.buf);
BTScanPosInvalidate(so->currPos); BTScanPosInvalidate(so->currPos);
return false; return false;
} }
} }
else else
{
blkno = opaque->btpo_next; blkno = opaque->btpo_next;
_bt_relbuf(rel, so->currPos.buf); _bt_relbuf(rel, so->currPos.buf);
} }
} }
}
else else
{ {
/* /*
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment