From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Neil Conway <neilc(at)postgresql(dot)org>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Do a pass of code review for the ALTER TABLE |
Date: | 2006-07-02 17:18:33 |
Message-ID: | 20060702171833.GC22974@surnet.cl |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Bruce Momjian wrote:
> Neil Conway wrote:
> > Log Message:
> > -----------
> > Do a pass of code review for the ALTER TABLE ADD INHERITS patch. Keep
> > the read lock we hold on the table's parent relation until commit.
> > Update equalfuncs.c for the new field in AlterTableCmd. Various
> > improvements to comments, variable names, and error reporting.
> >
> > There is room for further improvement here, but this is at least
> > a step in the right direction.
>
> Thanks, that is what was needed. The author obviously took the patch as
> far as he could, and we needed to adjust his XXX areas, rather than not
> apply the patch and have the code drifting.
Hmm, is this how we should do things? I mean, should I finish the
autovacuum parts of my relminxid patch, apply it, and then hope for
someone to fix the mistaeks? And if we don't see any failure in the
buildfarm, assume that all is well?
To me this is really the easiest way, but I have a hard time convincing
myself that I want to have it easy but break things in a way that nobody
notices. The other day when I typoed a commit to the 8.1 branch I was
all red in the face. I wonder what will happen if someone points to me
or Greg as causing major breakage somewhere, just because the patch was
applied in a hurry without careful review.
--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2006-07-02 18:50:18 | Re: [COMMITTERS] pgsql: Do a pass of code review for the |
Previous Message | Bruce Momjian | 2006-07-02 16:24:57 | Re: [COMMITTERS] pgsql: Do a pass of code review for the ALTER TABLE |
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2006-07-02 17:36:30 | Re: note about syntax for fillfactor patch |
Previous Message | Tom Lane | 2006-07-02 16:34:36 | Re: ADD/DROPS INHERIT (actually INHERIT / NO INHERIT) |