Re: Skytools committed without hackers discussion/review

From: Shane Ambler <pgsql(at)Sheeky(dot)Biz>
To: Magnus Hagander <magnus(at)hagander(dot)net>
Cc: Marko Kreen <markokr(at)gmail(dot)com>, "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com>, Michael Glaesemann <grzm(at)seespotcode(dot)net>, Bruce Momjian <bruce(at)momjian(dot)us>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Skytools committed without hackers discussion/review
Date: 2007-10-10 14:23:32
Message-ID: 470CE064.4080700@Sheeky.Biz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Magnus Hagander wrote:

> If it is this irreplacable killer feature, it should *not* be in contrib.
> It should be in the core backend, and we should be discussing if we can
> bend the rules for that. This is the proper forum for discussing that, so
> let's bring that question to the table.

+1 there, I don't think it should go into contrib just cause it was a
late entry. It really seems to be a matter of whether it gets into 8.3
or 8.4

> Our beta-1 is already fairly broken (the locale stuff on our most
> downloaded platform), so perhaps we should pull that one back, put this
> stuff in the backend, and try to get a beta2 out ASAP?
>

The question there is how long will it take to reach a decision of where
the patch belongs? (8.3 8.4 or contrib)

> Putting it in contrib "just because we were too late to put it in the
> backend, but it is reallyi really important for our users" just doesn't
> make sense.
>
+1

--

Shane Ambler
pgSQL(at)Sheeky(dot)Biz

Get Sheeky @ http://Sheeky.Biz

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2007-10-10 14:29:43 Re: Skytools committed without hackers discussion/review
Previous Message Bruce Momjian 2007-10-10 14:09:49 pgsql: Back out release changes that weren't ready for commit.

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2007-10-10 14:29:43 Re: Skytools committed without hackers discussion/review
Previous Message Dave Page 2007-10-10 14:15:50 Re: Locale + encoding combinations