Re: [PATCHES] WITH RECURSIVE updated to CVS TIP

From: Tatsuo Ishii <ishii(at)postgresql(dot)org>
To: david(at)fetter(dot)org
Cc: y-asaba(at)sraoss(dot)co(dot)jp, postgres(at)cybertec(dot)at, pgsql-patches(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES] WITH RECURSIVE updated to CVS TIP
Date: 2008-07-08 09:01:05
Message-ID: 20080708.180105.00468926.t-ishii@sraoss.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Here is the patches he made against CVS HEAD (as of today).

According to him followings are fixed with the patches:

- fix crush with DISTINCT
- fix creating VIEW
- fix the case when recursion plan has another recursion plan under it
- fix WITH RECURSIVE ...(..) SELECT ...WHERE.. returns wrong result
- fix inifinit recursion with OUTER JOIN

Not yet fixed:

- detect certain queries those are not valid acroding to the standard
- sort query names acording to the dependency
- planner always estimate 0 cost for recursion plans
--
Tatsuo Ishii
SRA OSS, Inc. Japan

> > - SQL:2008 に規定されているクエリ以外をエラーにする処理
> > - 依存関係の順番で評価するようにする仕組み
> > - プランナが常にコスト 0 で見積る
> >

> On Mon, Jul 07, 2008 at 04:22:21PM +0900, Yoshiyuki Asaba wrote:
> > Hi,
> >
> > > test=# explain select count(*)
> > > test-# from ( WITH RECURSIVE t(n) AS ( SELECT 1 UNION ALL
> > > SELECT DISTINCT n+1 FROM t )
> > > test(# SELECT * FROM t WHERE n < 5000000000) as t
> > > test-# WHERE n < 100;
> > > server closed the connection unexpectedly
> > > This probably means the server terminated abnormally
> > > before or while processing the request.
> > > The connection to the server was lost. Attempting reset: Failed.
> > > !> \q
> > >
> > > this one will kill the planner :(
> > > removing the (totally stupid) distinct avoids the core dump.
> >
> > Thanks. I've fixed on local repository.
>
> Asaba-san, do you have a patch against CVS HEAD or against the
> previous one?
>
> Cheers,
> David.
> --
> David Fetter <david(at)fetter(dot)org> http://fetter.org/
> Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
> Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
>
> Remember to vote!
> Consider donating to Postgres: http://www.postgresql.org/about/donate

Attachment Content-Type Size
recursive_query-8.patch.bz2 application/octet-stream 21.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message KaiGai Kohei 2008-07-08 09:08:49 Re: Proposal of SE-PostgreSQL patches [try#2]
Previous Message Zdenek Kotala 2008-07-08 08:22:32 Re: [WIP] patch - Collation at database level

Browse pgsql-patches by date

  From Date Subject
Next Message Simon Riggs 2008-07-08 12:16:31 Re: [PATCHES] WIP: executor_hook for pg_stat_statements
Previous Message Simon Riggs 2008-07-08 07:36:21 Re: [HACKERS] Userset logging