On Mon, 01 Dec 2003 13:32:10 -0500, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>Manfred Koizar <mkoi-pg@aon.at> writes:
>> comparetup_index() compares two IndexTuples. The structure
>> IndexTupleData consists basically of not much more than an ItemPointer,
>> and the patch is not much more than adding a comparison of two
>> ItemPointers. So how does the patch introduce a new low level
>> implementation dependency?
>
>Because it sorts on tuple position, which is certainly about as low
>level as you can get.
The patch affects only the sort during index creation. Mapping key
values to tuple positions is the sole purpose of an index. The notion
that an index should not care for tuple positions looks a bit strange to
me.
> More to the point, though, no evidence has been
>provided that this is a good idea.
The test script I posted with the patch shows that the patch produces
more efficient b-tree indices when there are lots of duplicates.
Servus
Manfred