Re: conditional dropping of columns/constraints - Mailing list pgsql-hackers

From Andres Freund
Subject Re: conditional dropping of columns/constraints
Date
Msg-id 4A02BEB4.7090600@anarazel.de
Whole thread Raw
In response to Re: conditional dropping of columns/constraints  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: conditional dropping of columns/constraints  (Bernd Helmle <mailings@oopsware.de>)
List pgsql-hackers
Hi Tom, hi all,

On 05/06/2009 11:43 PM, Tom Lane wrote:
> Andres Freund<andres@anarazel.de>  writes:
>> As this is my first patch to PG I am happy with most sort of feedback.
> Please add your patch to the commit-fest queue here:
> http://wiki.postgresql.org/wiki/CommitFestInProgress
Will do so, after this email has arrived.

> Since we are still busy with 8.4 beta, it's unlikely that anyone will
> take a close look until the next commit fest begins.  FWIW, I took a
> very fast look through the patch and thought it was at least touching
> the right places, except I think you missed equalfuncs.c.  (It'd be a
> good idea to grep for all uses of AlterTableCmd struct to see if you
> missed anything else.)  I don't have time now to look closer or do any
> testing.
Ok, fixed that place (stupid me, I had seen that it is used there and
forgot about it) and found no other places.

Thanks,

Andres

Attachment

pgsql-hackers by date:

Previous
From: Gregory Stark
Date:
Subject: Re: Serializable Isolation without blocking
Next
From: Heikki Linnakangas
Date:
Subject: Re: ECPG, two varchars with same name on same line