Re: Values list-of-targetlists patch for comments (was Re: - Mailing list pgsql-patches

From Joe Conway
Subject Re: Values list-of-targetlists patch for comments (was Re:
Date
Msg-id 44C4F625.2050304@joeconway.com
Whole thread Raw
In response to Re: Values list-of-targetlists patch for comments (was Re: [HACKERS] 8.2 features?)  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Values list-of-targetlists patch for comments (was Re: [HACKERS] 8.2 features?)
List pgsql-patches
Tom Lane wrote:
>>
>>>There are basically two ways you could go about this:
>>>1. Make a new jointree leaf node type to represent a VALUES construct,
>>>and dangle the list of lists of expressions off that.
>>>2. Make a new RangeTblEntry type to represent a VALUES construct, and
>>>just put a RangeTblRef to it into the jointree.  The expressions
>>>dangle off the RangeTblEntry.
>
> You seem to have done *both*, which is certainly not what I had in mind.
> I'd drop the RangeTblEntry changes, I think.

Good feedback -- thanks! But without the RTE, how would VALUES in the
FROM clause work? Or should I just drop that part and focus on just the
InsertStmt case?

Joe

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: Values list-of-targetlists patch for comments (was Re: [HACKERS] 8.2 features?)
Next
From: Tom Lane
Date:
Subject: Re: Resurrecting per-page cleaner for btree