Skip site navigation (1) Skip section navigation (2)

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: (view raw, whole thread or download thread mbox)
Lists: pgsql-rrreviewers

* 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:
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. :)



In response to


pgsql-rrreviewers by date

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

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group