Re: [PATCHES] WITH RECUSIVE patches 0717

From: "Erik" <er(at)xs4all(dot)nl>
To: "Tatsuo Ishii" <ishii(at)postgresql(dot)org>
Cc: pgsql-patches(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org, y-asaba(at)sraoss(dot)co(dot)jp
Subject: Re: [PATCHES] WITH RECUSIVE patches 0717
Date: 2008-07-18 17:30:24
Message-ID: 12057.156.83.1.19.1216402224.squirrel@webmail.xs4all.nl
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

On Fri, July 18, 2008 03:41, Tatsuo Ishii wrote:
>> > Here is the lastest WITH RECURSIVE patches against CVS HEAD created by
>> > Yoshiyuki Asaba and minor corrections by Tatsuo Ishii.
>>
>> I tried this patch vs. CVS HEAD used my usual configure option with
>> only --with-prefix set, then tried to make, and got:
>>
>> make[3]: *** No rule to make target `parse_cte.o', needed by `objfiles.txt'.
>> Stop.
>> make[3]: Leaving directory `/home/shackle/pgsql/src/backend/parser'
>> make[2]: *** [parser-recursive] Error 2
>> make[2]: Leaving directory `/home/shackle/pgsql/src/backend'
>> make[1]: *** [all] Error 2
>> make[1]: Leaving directory `/home/shackle/pgsql/src'
>> make: *** [all] Error 2
>>
>> Is there something missing?
>
> Oops. I forgot to include patches against newly added files. Please
> try included patches.
>

This crashes the backend:

WITH RECURSIVE t(n) AS (
VALUES (1)
UNION ALL
SELECT n+1 FROM t WHERE n < 5 ORDER BY 1
)
SELECT n FROM t;

apparently because of the ORDER BY 1

( ORDER BY t.n will just error out )

Compiled with:

./configure \
--prefix=${install_dir} \
--with-pgport=${pgport} \
--quiet \
--enable-depend \
--enable-cassert \
--enable-debug \
--with-openssl

hth

Erik Rijkers

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2008-07-18 20:26:07 pgsql: Adjust things so that the query_string of a cached plan and the
Previous Message Kenneth Marshall 2008-07-18 17:23:14 Re: [PATCH]-hash index improving

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2008-07-18 17:33:17 Re: doc patch - archive/restore_command on windows
Previous Message David Fetter 2008-07-18 15:13:08 Re: [PATCHES] WITH RECUSIVE patches 0717