Re: ALTER TABLE RENAME fix - Mailing list pgsql-patches

From Brent Verner
Subject Re: ALTER TABLE RENAME fix
Date
Msg-id 20011112143855.A2446@rcfile.org
Whole thread Raw
In response to Re: ALTER TABLE RENAME fix  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: ALTER TABLE RENAME fix
Re: ALTER TABLE RENAME fix
List pgsql-patches
On 12 Nov 2001 at 18:28 (+0100), Peter Eisentraut wrote:
| Brent Verner writes:
|
| > These patches fix the problem where an
| >
| >   ALTER TABLE <table> RENAME <oldcolumn> TO <newcolumn>
| >
| > did not update the RI_ triggers if the oldcolumn was referenced in
| > a RI constraint.
|
| Instead of trying to fix this, how about making the RI triggers not use
| the column names in the first place.  (Instead they should use the oid of
| the table and the attnums.)

I agree with you.  Two things led me toward the current approach
(modifying tgargs)
  1) it would require fewer changes to get correct behavior, which
     was important as late in beta as I started and
  2) I had /some/ idea of how to get it done ;-)

I'll begin to look at what would be required to do it the right
way for 7.3.

cheers.
  brent

--
"Develop your talent, man, and leave the world something. Records are
really gifts from people. To think that an artist would love you enough
to share his music with anyone is a beautiful thing."  -- Duane Allman

pgsql-patches by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: Patch for Makefile race against current cvs
Next
From: Bruce Momjian
Date:
Subject: Re: ALTER TABLE RENAME fix