Re: [HACKERS] btree_gist (was: CommitFest progress - or lack thereof)

Robert Haas wrote:
> On Fri, Feb 4, 2011 at 12:02 PM, Oleg Bartunov <oleg@sai.msu.su> wrote:
> > Aha,
> >
> > Teodor sent it to the list Dec 28, see
> > http://archives.postgresql.org/message-id/4D1A1677.80300%40sigaev.ru
> >
> > After a month I didn't see any activity on this patch, so I I added it to
> > https://commitfest.postgresql.org/action/patch_view?id=350 Jan 21
> >
> > Now, I realised it was too late. Added to current commitfest.
>
> I think this patch missed the deadline for the current CommitFest.
> It's true that it was posted to the list in time, but it's just
> madness to think we can do review in a meaningful way and get done in
> a reasonable time if every patch that's ever been posted is fair game
> to be added to the CommitFest at any point.  I believe it's a
> generally accepted principle that adding things to the CommitFest
> properly is the submitter's responsibility.
>
> That having been said, this looks like a fairly mechanical change to a
> contrib module that you and Teodor wrote.  So I'd say if you guys are
> confident that it's correct, go ahead and commit.  If you need it
> reviewed, or if you can't commit it in the next week or so, I think
> it's going to have to wait for 9.2.

If you need a +1, agreed.

--
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + It's impossible for everything to be true. +