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(-)
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. |