diff options
| author | Robert Haas <rhaas@postgresql.org> | 2012-08-30 13:05:45 -0400 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2012-08-30 13:09:07 -0400 |
| commit | c8ba697a4bdb934f0c51424c654e8db6133ea255 (patch) | |
| tree | 8d8ed4a104a58708f3b0b51d9288e62b0f549f4b /src/backend/utils/adt/array_selfuncs.c | |
| parent | d1a4db8d25ec53fd17e99168bc5efa0b16ef6fed (diff) | |
| download | postgresql-c8ba697a4bdb934f0c51424c654e8db6133ea255.tar.gz postgresql-c8ba697a4bdb934f0c51424c654e8db6133ea255.zip | |
Fix logic bug in gistchoose and gistRelocateBuildBuffersOnSplit.
Every time the best-tuple-found-so-far changes, we need to reset all
the penalty values in which_grow[] to the penalties for the new best
tuple. The old code failed to do this, resulting in inferior index
quality.
The original patch from Alexander Korotkov was just two lines; I took
the liberty of fleshing that out by adding a bunch of comments that I
hope will make this logic easier for others to understand than it was
for me.
Diffstat (limited to 'src/backend/utils/adt/array_selfuncs.c')
0 files changed, 0 insertions, 0 deletions