Re: Bad Build

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Jan Wieck <janwieck(at)yahoo(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Neil Conway <nconway(at)klamath(dot)dyndns(dot)org>, Rod Taylor <rbt(at)zort(dot)ca>, Hackers List <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Bad Build
Date: 2002-03-07 18:05:51
Message-ID: 200203071805.g27I5pa14970@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-cygwin pgsql-hackers

Jan Wieck wrote:
> Bruce Momjian wrote:
> > Tom Lane wrote:
> > > By my count the breakage from the DOMAIN patch is:
> > > 1 shift/reduce conflict in gram.y
> > > 3 gcc warnings (at least one being obviously a bug)
> > > 1 core dump during regression tests
> > >
> > > Bruce, what in the heck were you doing applying this patch? You knew
> > > darn well it had not been meaningfully reviewed. Not bothering to check
> > > for compile problems or regression failures before applying is
> > > unforgivably sloppy work. I don't blame the submitter; I blame you,
> > > who should have known better.
> >
> > You can blame me all you want. That was in the queue, and no one
> > objected, so I did my best. If you don't want to forgive me, don't.
> >
> > In fact, this whole indignation thing is starting to tire me. This is an
> > open-source project. People do the best they can. Just make the best of
> > it and move on.
> >
> > If this is the worst that has happened from my applying all those back
> > patches, I am happy.
>
> Sorry Bruce, but just because your patch queue is very long
> due to the delays in the 7.2 release cycle is no excuse to
> work sloppy now. Rushing things in is not the solution.

No, this was not in a very old patch. First patch appeared late
February, there was discussion, a second patch appeared early March, and
there was no discussion on it. The patch had a file of sample queries,
and someone else had even submitted a psql patch based on the feature.
So, actually, it looked pretty good.

In fact, the patch did have a compile problem when applied because it
used our commandTag that isn't used anymore in that place, so I fixed
it.

However, that wasn't really my issue. I am happy to back out anything,
and have does so for both patches listed above. I will contact the
authors and get an updated version that we can discuss and request more
testing.

My issue is that trying to blame someone isn't the proper way to address
these issues.

--
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-cygwin by date

  From Date Subject
Next Message Rod Taylor 2002-03-07 18:18:02 Re: Bad Build
Previous Message Fernando Nasser 2002-03-07 17:48:38 Re: Bad Build

Browse pgsql-hackers by date

  From Date Subject
Next Message Rod Taylor 2002-03-07 18:18:02 Re: Bad Build
Previous Message Fernando Nasser 2002-03-07 17:48:38 Re: Bad Build