Re: top-level DML under CTEs - Mailing list pgsql-hackers

From Robert Haas
Subject Re: top-level DML under CTEs
Date
Msg-id AANLkTikA=XOsGmV+Bq-TDdMj-OpxPSpCVk0GV6_GNH5G@mail.gmail.com
Whole thread Raw
In response to Re: top-level DML under CTEs  (Marko Tiikkaja <marko.tiikkaja@cs.helsinki.fi>)
List pgsql-hackers
On Tue, Sep 14, 2010 at 4:28 PM, Marko Tiikkaja
<marko.tiikkaja@cs.helsinki.fi> wrote:
> On 2010-09-14 10:51 PM, Tom Lane wrote:
>>
>> Hitoshi Harada<umi.tanuki@gmail.com>  writes:
>>>
>>> 2010/9/15 Marko Tiikkaja<marko.tiikkaja@cs.helsinki.fi>:
>>>>
>>>> In the email you referred to, Tom was concerned about the case where
>>>> these
>>>> WITH lists have different RECURSIVE declarations.  This patch makes both
>>>> RECURSIVE if either of them is.  I can think of cases where that might
>>>> lead
>>>> to surprising behaviour, but the chances of any of those happening in
>>>> real
>>>> life seem pretty slim.
>>
>>> Does that cause surprising behavior?
>>
>> My recollection is that whether a CTE is marked RECURSIVE or not affects
>> its scope of visibility, so that confusing the two cases can result in
>> flat-out incorrect parser behavior.
>
> The worst I can think of is:
>
> CREATE TABLE foo(a int);
>
> WITH t AS (SELECT * FROM foo)
> INSERT INTO bar
> WITH RECURSIVE foo (SELECT 1 AS a)
> SELECT * FROM t;
>
> t will actually be populated with the results of the CTE, not the table foo.

Unless I'm confused, that seems pretty clearly wrong.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company


pgsql-hackers by date:

Previous
From: Domas Mituzas
Date:
Subject: Re: Perf regression in 2.6.32 (Ubuntu 10.04 LTS)
Next
From: Bernd Helmle
Date:
Subject: Re: ALTER TABLE...ALTER COLUMN vs inheritance