From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Stephen Frost <sfrost(at)snowman(dot)net>, Jaime Casanova <jcasanov(at)systemguards(dot)com(dot)ec>, pgsql-rrreviewers(at)postgresql(dot)org |
Subject: | Re: need more reviewers for index changes |
Date: | 2009-07-21 18:54:46 |
Message-ID: | 937d27e10907211154n6f08355ese2bea31912ccfab0@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-rrreviewers |
On 7/21/09, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Tue, Jul 21, 2009 at 6:40 AM, Stephen Frost<sfrost(at)snowman(dot)net> wrote:
>> Robert,
>>
>> * Robert Haas (robertmhaas(at)gmail(dot)com) wrote:
>>> It probably makes sense to start with indexam api changes, since the
>>> other one depends on that one.
>>
>> I've looked the patch over, but the problem is that the patch doesn't do
>> terribly much by itself, and Tom's already commented on things he
>> doesn't like about it. Alot of the patch here:
>> http://archives.postgresql.org/message-id/4A5ADFE6.6060507@enterprisedb.com
>> is changing heap_hot_search_buffer to have heapTuple passed in as a
>> pointer rather than a local var, which makes for many '.' to '->'
>> changes. The rest is just splitting index_getnext into two pieces.
>
> Ah, yes, I see your point. So I think this is actually waiting on
> author. We should poke Heikki.
I believe he's on vacation.
--
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Stephen Frost | 2009-07-21 21:41:41 | Re: need more reviewers for index changes |
Previous Message | Robert Haas | 2009-07-21 17:54:42 | Re: CF 2009-07: initial reviewing assignments |