Re: ANSI Compliant Inserts

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Hiroshi Inoue <Inoue(at)tpf(dot)co(dot)jp>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Rod Taylor <rbt(at)zort(dot)ca>, pgsql-patches(at)postgresql(dot)org
Subject: Re: ANSI Compliant Inserts
Date: 2002-04-16 00:47:39
Message-ID: 200204160047.g3G0leX15112@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Hiroshi Inoue wrote:
> > I meant to say there was no question we wanted this item fixed, not that
> > there was no need for implementation discussions.
> >
> > In summary, code changes have three stages:
> >
> > o Do we want this feature?
> > o How do we want the feature to behave?
> > o How do we want the feature implemented?
> >
> > Tom was complaining because the patch appeared without enough discussion
> > on these items. However, from my perspective, this is really trying to
> > micromanage the process. When people post patches, we aren't forced to
> > apply them.
>
> But shouldn't someone check the patch ?
> If the patch is small, making the patch seems
> the simplest way for anyone but if the patch
> is big, it seems painful for anyone to check
> the patch. If no one checks the patch, would
> we apply the patch blindly or reject it ?

Of course, we would review any patch before application. I guess the
full path is:

o Do we want this feature?
o How do we want the feature to behave?
o How do we want the feature implemented?
o Submit patch
o Review patch
o Apply patch

I assume your point is that people shouldn't send in big patches
without going through the discussion first. Yes, that is ideal, but if
they don't, we just discuss it after the patch appears, and then decide
if we want to apply it or ask for modifications.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2002-04-16 00:54:26 Re: 7.2 crash...
Previous Message Hiroshi Inoue 2002-04-16 00:15:23 Re: ANSI Compliant Inserts

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2002-04-16 01:37:12 Re: YADP - Yet another Dependency Patch
Previous Message Tom Lane 2002-04-16 00:40:46 Re: default values for views