Re: TODO question

> Bruce Momjian writes:
> > Sure, ask away.  We will do our best.  In fact, adding a new node is
> > pretty tricky.  There is a developer's FAQ item about it, number 7.  I
> > assume you read that already.
> 
> hm...do you mean the current version of FAQ on the web? there are 2 seventh
> items in the table of contents, both jumping to
> ----------------------------------------snip--------------------------------
> -
> 7) I just added a field to a structure. What else should I do?
> The structures passing around from the parser, rewrite, optimizer, and
> executor require quite a bit of support. Most structures have support
> routines in src/backend/nodes used to create, copy, read, and output those
> structures. Make sure you add support for your new field to these files.
> Find any other places the structure may need code for your new field. mkid
> is helpful with this (see above).
> 
> ----------------------------------------snap--------------------------------
> -
> 
> Is that what you mean? It confuses me a bit, surely because I'm new here...
> I didn't change any existing struct, and coudn't find any struct I which
> could grow bacause of my new parsenode type. The type-enum contains a
> corresponding range of values (I think, it was smth. starting with 700 upto
> ...) - I just added my new type here.
> Or do I use a wrong copy of FAQ?

Yes, that is the one.  The steps for adding an entry to an existing
structure is similar to add an entirely new one.

> > I may be able to take you patch and add the needed node support stuff,
> > and send the patch back to you so you can continue on it.
> 
> thanx. please, feel free - I attached my last patch to this email.
> I see, that my patch provides the needed operation, but maybe it's not
> enough and could cause some side effects. I'm looking forward to your
> modifications (I hope it doesn't keep you from your current work, at least
> not crucial). What I'm just interested in is, if my changes would be a
> subset of your code or if what I did is an absolute b*-sh* ,)

I think it would be a subset.  I haven't looked at it closely yet,
though.

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026