Re: GiST insert algorithm rewrite - Mailing list pgsql-hackers

From Robert Haas
Subject Re: GiST insert algorithm rewrite
Date
Msg-id 37B49EC9-06DD-404E-BCAB-E73D9DBA4EB5@gmail.com
Whole thread Raw
In response to Re: GiST insert algorithm rewrite  (Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>)
List pgsql-hackers
On Dec 3, 2010, at 4:54 PM, Heikki Linnakangas <heikki.linnakangas@enterprisedb.com> wrote:
> Here's an updated patch.

How carefully have you perf-tested this?

> On closer look, supporting the invalid tuples in scans was trivial, so I kept that after all. So you can still query
anindex with invalid tuples. If an insert encounters one, you get an error, and VACUUM emits a LOG message on any such
tuples.

Cool.

> There's one bug remaining that I found during testing. If you crash, leaving an incomplete split behind, and then
vacuumthe table removing all the aborted tuples from the pages, it's possible that you end up with a completely empty
pagethat has no downlink yet. The code to complete incomplete splits doesn't cope with that at the moment - it doesn't
knowhow to construct a parent key for a child that has no tuples. 

I think we can live with this.
>


...Robert

pgsql-hackers by date:

Previous
From: Vaibhav Kaushal
Date:
Subject: Requirement for the Buffer manager to load multiple pages at once
Next
From: Tom Lane
Date:
Subject: Note about KNNGIST limitation: no mark/restore