Re: [PATCHES] COPY view - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [PATCHES] COPY view
Date
Msg-id 20060823161042.GS1963@alvh.no-ip.org
Whole thread Raw
In response to Re: [PATCHES] COPY view  (Zoltan Boszormenyi <zboszor@dunaweb.hu>)
Responses Re: [PATCHES] COPY view  (Zoltan Boszormenyi <zboszor@dunaweb.hu>)
List pgsql-hackers
Zoltan Boszormenyi wrote:

> OK, here's my current version. The reference leak is fixed.
> But as my testcase shows, it only works for single selects
> currently. The parser accepts it but COPY doesn't produce
> the expected output. Please, suggest a solution.

I'm not sure I agree with the approach of creating a fake "SELECT * FROM
foo" in analyze.c in the relation case and passing it back to the parser
to create a Query node.  That's not there in the original code and you
shouldn't need it.  Just let the case where COPY gets a relation
continue to handle it as it does today, and add a separate case for the
SELECT.

That doesn't help you with the UNION stuff though.

--
Alvaro Herrera                                http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

pgsql-hackers by date:

Previous
From: Zoltan Boszormenyi
Date:
Subject: Re: [PATCHES] COPY view
Next
From: Zoltan Boszormenyi
Date:
Subject: Re: [PATCHES] COPY view