From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Martijn van Oosterhout <kleptog(at)svana(dot)org>, pgsql-patches(at)postgresql(dot)org |
Subject: | Re: Clear out the reminants of EXTEND INDEX |
Date: | 2001-07-16 18:04:31 |
Message-ID: | 200107161804.f6GI4Vm23525@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
> > diff -u -r1.91 nodes.h
> > --- src/include/nodes/nodes.h 2001/06/19 22:39:12 1.91
> > +++ src/include/nodes/nodes.h 2001/07/16 09:35:35
> > @@ -158,7 +158,6 @@
> > T_DropStmt,
> > T_TruncateStmt,
> > T_CommentStmt,
> > - T_ExtendStmt,
> > T_FetchStmt,
> > T_IndexStmt,
> > T_ProcedureStmt,
>
> This is not good --- we try to avoid renumbering existing node types.
> (Not for any essential reason, perhaps, but why risk breaking anything?)
> My practice when removing a node type is to change the nodes.h entry
> like this:
>
> T_RemoveOperStmt,
> T_RemoveStmt_XXX, /* not used anymore; tag# available */
> T_RenameStmt,
>
> Adding the _XXX is sufficient to ensure that any missed uses of the
> typecode in the source will yield compile errors.
>
> Otherwise it looks OK.
I always rip that stuff out. We do an initdb for every release. What
could break by renumbering those nodes?
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2001-07-16 18:55:31 | Re: Clear out the reminants of EXTEND INDEX |
Previous Message | Tom Lane | 2001-07-16 15:58:04 | Re: Clear out the reminants of EXTEND INDEX |