DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma;
	h=mime-version:in-reply-to:references:date:message-id:subject:from:to
	:cc:content-type;
	b=J2Q0N6Thg0ukP/p2QkOO8nLeAhDmfYEX7dthbgjglArlK1uBqn7R3Kdf93V0mIcJLu
	1t1Iu4dIDHz0a0BiW5/MsoO7IUVbBtPMNr+ZT3zWHeiImXmyoLLFTo8gzXhLhFKA1kPw
	oQcovy594TFCXfzKltN8+36OLi/DNu0FH59e4=
MIME-Version: 1.0
In-Reply-To: <alpine.DEB.2.00.1003151548180.1887@aragorn.flymine.org>
References: <201002252342.o1PNgtG07127@momjian.us>
	<alpine.DEB.2.00.1003151548180.1887@aragorn.flymine.org>
Date: Mon, 15 Mar 2010 21:58:57 -0400
Message-ID: <603c8f071003151858o4d73d856n3a2d6a1d9a7da03@mail.gmail.com>
Subject: Re: GiST index performance
From: Robert Haas <robertmhaas@gmail.com>
To: Matthew Wakeling <matthew@flymine.org>
Cc: Bruce Momjian <bruce@momjian.us>, pgsql-performance@postgresql.org
Content-Type: text/plain; charset=ISO-8859-1

On Mon, Mar 15, 2010 at 11:58 AM, Matthew Wakeling <matthew@flymine.org> wrote:
> On Thu, 25 Feb 2010, Bruce Momjian wrote:
>>
>> Was there every any conclusion on this issue?
>
> Not really. Comments inline:
>
>> Matthew Wakeling wrote:
>>>
>>> Revisiting the thread a month back or so, I'm still investigating
>>> performance problems with GiST indexes in Postgres.
>>>
>>> Looking at http://wiki.postgresql.org/wiki/PostgreSQL_8.4_Open_Items I'd
>>> like to clarify the contrib/seg issue. Contrib/seg is vulnerable to
>>> pathological behaviour which is fixed by my second patch, which can be
>>> viewed as complete. Contrib/cube, being multi-dimensional, is not
>>> affected
>>> to any significant degree, so should not need alteration.
>
> This issue is addressed by my patch, which AFAIK noone has reviewed.
> However, that patch was derived from a patch that I applied to bioseg, which
> is itself a derivative of seg. This patch works very well indeed, and gave
> an approximate 100 times speed improvement in the one test I ran.
>
> So you could say that the sister patch of the one I submitted is tried and
> tested in production.

We rely fairly heavily on the commitfest app to track which patches
need review; perhaps it would be good to add it here.

https://commitfest.postgresql.org/action/commitfest_view/open

I seem to recall thinking that this patch wasn't ready to apply for
some reason, but I'm not sure why I thought that.

...Robert