Re: need more reviewers for index changes

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: 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 10:40:02
Message-ID: 20090721104001.GA23840@tamriel.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-rrreviewers

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.

To be honest, I'm not convinced I'm qualified to review this patch
anyway, but I was giving it a go. Of course, it strikes me that this
patch has had a fair bit of feedback from committers already, and
anything I try to add will likely end up being wrong anyway. :)

Thanks,

Stephen

In response to

Responses

Browse pgsql-rrreviewers by date

  From Date Subject
Next Message Robert Haas 2009-07-21 17:06:56 Re: CF 2009-07: initial reviewing assignments
Previous Message Robert Haas 2009-07-21 05:12:33 Re: need more reviewers for index changes