Re: WIP: store additional info in GIN index

From: Alexander Korotkov <aekorotkov(at)gmail(dot)com>
To: Craig Ringer <craig(at)2ndquadrant(dot)com>
Cc: Tomas Vondra <tv(at)fuzzy(dot)cz>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: WIP: store additional info in GIN index
Date: 2013-03-03 17:29:57
Message-ID: CAPpHfdv0i1pbqDd9fhE5=+5nUFZiCa+qdQ98inwPTcsyAZyxPA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Mar 3, 2013 at 6:53 PM, Craig Ringer <craig(at)2ndquadrant(dot)com> wrote:

> The GIN changes don't seem to have progressed in some time, and some of
> the most recent activity
> (http://archives.postgresql.org/message-id/50BFF89A.7080908@fuzzy.cz)
> suggests unconvincing test results.
>

Actually, _most_ recent acitivity showing inverse
http://www.postgresql.org/message-id/50D79861.3030303@fuzzy.cz
However, this patch itself is not expected to give significant changes in
search speed. Testing results with both double acceleration and slowdown
looks strange for me. I can't either reproduce it or explain.

Is this work considered to be a dead-end - a good idea that didn't work
> out in practice? Or do you think it still has merit and can be made
> useful and ready for inclusion?
>

This patch is only first of future serie of GIN improvements patches. It
doesn't change anything significant in search, only in storage. This time
we are working on design of rest of patches in order to put them on the
consideration. This lead to lack of attention to this patch.

Given the activity level I would like to bounce this patch, either as
> "returned with feedback" if you want to take another go at it post-9.3,
> or as "rejected" if you think the idea won't go anywhere. Please let me
> know how you think it looks.
>

"Returned with feedback", definitely.

------
With best regards,
Alexander Korotkov.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Korotkov 2013-03-03 17:42:23 Re: SP-GiST for ranges based on 2d-mapping and quad-tree
Previous Message Dean Rasheed 2013-03-03 17:06:36 Re: Materialized views WIP patch