Your patch has been added to the PostgreSQL unapplied patches list at:
http://candle.pha.pa.us/cgi-bin/pgpatches
I will try to apply it within the next 48 hours.
>
> Here is another version of the multiple insert patch that contains
> modifications as per Tom's post on the 24th.
>
> I had though about the integration of targetList and valuesList into
> something like targetLists, but I didn't do it because I knew there
> would need to be modifications in a lot of places and because the
> difference between targetList and valuesList seemed clear. I gave Tom's
> suggestion a shot, though, and this is the first result. There are quite
> a number of functions that take a single targetList as a parameter. I
> didn't change those because I think it's more trouble that it's worth
> right now. If those modifications are desired, they should wait until
> the redesign of the the querytree structure, IMHO. But targetList ->
> targetLists is done.
>
> There are a couple of places where I made an assertion that the length
> of a Query node's targetLists member must be one that I'm not sure about
> (there are comments by those assertions). Suggestions on those would be
> appreciated.
>
> There are also a couple of places where I need to do a check to see if a
> query's targetLists is NIL and do assignments based on the result of
> that check. Would it be better to for all Query nodes that currently
> have a NIL targetLists coming out of transfromXYZ to have a targetLists
> member with one NIL targetList. I don't really think so (it's could be
> confusing), but I'm just wondering what the thoughts of others are.
>
> Comments?
>
> Liam
>
> --
> Liam Stewart :: Red Hat Canada, Ltd. :: liams@redhat.com
[ Attachment, skipping... ]
[ Attachment, skipping... ]
[ Attachment, skipping... ]
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org
--
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, Pennsylvania 19026
