Re: [patch] ALTER RENAME and indexes

On 08 Oct 2001 at 14:43 (-0400), Tom Lane wrote:
| Brent Verner <brent(at)rcfile(dot)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