Re: [HACKERS] Unusable SP-GiST index - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [HACKERS] Unusable SP-GiST index
Date
Msg-id 1010.1483146277@sss.pgh.pa.us
Whole thread Raw
In response to [HACKERS] Unusable SP-GiST index  (Vik Fearing <vik@2ndquadrant.fr>)
Responses Re: [HACKERS] Unusable SP-GiST index  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Vik Fearing <vik@2ndquadrant.fr> writes:
> While trying to find a case where spgist wins over btree for text, I
> came across the following behavior which I would consider a bug:

> CREATE TABLE texts (value text);
> INSERT INTO texts SELECT repeat('a', (2^20)::integer);
> CREATE INDEX ON texts USING spgist (value);
> SET enable_seqscan = off;
> TABLE texts;

> That produces:
> ERROR:  index row requires 12024 bytes, maximum size is 8191

Hmm ... it's not really SP-GiST's fault.  This query is trying to do
an index-only scan, and the API defined for that requires the index
to hand back an IndexTuple, which is of (very) limited size.
SP-GiST is capable of dealing with values much larger than one page,
but there's no way to hand them back through that API.

Maybe we should redefine the API as involving a TupleTableSlot that
the AM is supposed to fill --- basically, moving StoreIndexTuple
out of the common code in nodeIndexonlyscan.c and requiring the AM
to do that work.  The possible breakage of third-party code is a
bit annoying, but there can't be all that many third-party AMs
out there yet.
        regards, tom lane



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: [HACKERS] Potential data loss of 2PC files
Next
From: Tom Lane
Date:
Subject: Re: [HACKERS] Unusable SP-GiST index