]> granicus.if.org Git - postgresql/commit
Back-patch recent fixes for gistchoose and gistRelocateBuildBuffersOnSplit.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 31 Aug 2012 03:48:27 +0000 (23:48 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 31 Aug 2012 03:48:27 +0000 (23:48 -0400)
commit3d27e4dfab80e36c826c1c55dbb6b91576e5ccc3
tree0b6d342d8932d349591e1215773bd48d3f13cd92
parent588fb3f01f7159990a9bfabb578615c3c32ffc6e
Back-patch recent fixes for gistchoose and gistRelocateBuildBuffersOnSplit.

This back-ports commits c8ba697a4bdb934f0c51424c654e8db6133ea255 and
e5db11c5582b469c04a11f217a0f32c827da5dd7, which fix one definite and one
speculative bug in gistchoose, and make the code a lot more intelligible as
well.  In 9.2 only, this also affects the largely-copied-and-pasted logic
in gistRelocateBuildBuffersOnSplit.

The impact of the bugs was that the functions might make poor decisions
as to which index tree branch to push a new entry down into, resulting in
GiST index bloat and poor performance.  The fixes rectify these decisions
for future insertions, but a REINDEX would be needed to clean up any
existing index bloat.

Alexander Korotkov, Robert Haas, Tom Lane
src/backend/access/gist/gistutil.c