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

Peripheral Links

Header And Logo

PostgreSQL
| The world's most advanced open source database.

Site Navigation

Search archives
  Advanced Search

Re: need more reviewers for index changes


  • From: Robert Haas <robertmhaas(at)gmail(dot)com>
  • To: Stephen Frost <sfrost(at)snowman(dot)net>
  • 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: Tue, 21 Jul 2009 13:08:55 -0400
  • Message-id: <603c8f070907211008j40bfe769s2ca7f3d025dc935a@mail.gmail.com> <text/plain>

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(dot)6060507(at)enterprisedb(dot)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.

...Robert



Home | Main Index | Thread Index

Privacy Policy | About PostgreSQL
Copyright © 1996 – 2012 PostgreSQL Global Development Group