Re: [patch] ALTER RENAME and indexes - Mailing list pgsql-hackers

From Brent Verner
Subject Re: [patch] ALTER RENAME and indexes
Date
Msg-id 20011008201847.A7672@rcfile.org
Whole thread Raw
In response to Re: [patch] ALTER RENAME and indexes  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [patch] ALTER RENAME and indexes
List pgsql-hackers
On 08 Oct 2001 at 14:43 (-0400), Tom Lane wrote:
| Brent Verner <brent@rcfile.org> writes:
| > wooohoo!!!  Of course, it would be best if someone else looked this
| > code over, because I get the feeling there is an easier way to get
| > this done.
| 
| No, that's about right, except that you forgot one step: you shouldn't
| try to update column names of functional indexes, since their columns
| are named after the function not the column.  If the function name
| happened to match the column name then you'd have applied an erroneous
| renaming.
| 
| Fixed and applied to CVS.

Thanks for fixing that problem.  Out of curiousity, did you put that
code in the renameatt() function for any reason, or is it just a 
style thing?  (I ask in hopes of getting closer to submitting 
code/patches that you'd not have to rewrite to apply ;-) )

Thanks. 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-hackers by date:

Previous
From: "Serguei Mokhov"
Date:
Subject: Re: Place of PO files for NLS (was Re: [PATCHES] PG_DUMP NLS (Russian))
Next
From: KobayashiJunichi
Date:
Subject: PL/pgSQL doc sample bug? (in 7.1.3)