pgsql: Fix GiST buffering build bug, which caused "failed to re-find pa - Mailing list pgsql-committers

From Heikki Linnakangas
Subject pgsql: Fix GiST buffering build bug, which caused "failed to re-find pa
Date
Msg-id E1T1wuA-0001Po-8y@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix GiST buffering build bug, which caused "failed to re-find parent" errors.

We use a hash table to track the parents of inner pages, but when inserting
to a leaf page, the caller of gistbufferinginserttuples() must pass a
correct block number of the leaf's parent page. Before gistProcessItup()
descends to a child page, it checks if the downlink needs to be adjusted to
accommodate the new tuple, and updates the downlink if necessary. However,
updating the downlink might require splitting the page, which might move the
downlink to a page to the right. gistProcessItup() doesn't realize that, so
when it descends to the leaf page, it might pass an out-of-date parent block
number as a result. Fix that by returning the block a tuple was inserted to
from gistbufferinginserttuples().

This fixes the bug reported by Zdeněk Jílovec.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/89911b3ab848915dd64d21ca3d3537470c93f89c

Modified Files
--------------
src/backend/access/gist/gist.c      |   33 ++++++++++++++++++++++++++-------
src/backend/access/gist/gistbuild.c |   19 +++++++++++++------
src/include/access/gist_private.h   |    3 ++-
3 files changed, 41 insertions(+), 14 deletions(-)


pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgsql: Fix GiST buffering build bug, which caused "failed to re-find pa
Next
From: dpage@pgfoundry.org (Dave Page)
Date:
Subject: pginstaller - pginst: Update for 8.3.20