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

From: Heikki Linnakangas <heikki(dot)linnakangas(at)iki(dot)fi>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix GiST buffering build bug, which caused "failed to re-find pa
Date: 2012-08-16 10:02:18
Message-ID: E1T1wuA-0001Po-8y@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Dave Page 2012-08-16 10:43:35 pginstaller - pginst: Update for 8.3.20
Previous Message Bruce Momjian 2012-08-16 03:57:33 pgsql: Fix SGML markup; missing tag.