Re: [PATCHES] WITH RECURSIVE updated to CVS TIP

From: David Fetter <david(at)fetter(dot)org>
To: Tatsuo Ishii <ishii(at)postgresql(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-14 23:04:45
Message-ID: 20080714230445.GT14063@fetter.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

On Tue, Jul 08, 2008 at 06:01:05PM +0900, Tatsuo Ishii wrote:
> 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

Great!

I've patched psql for some partial support of WITH [RECURSIVE].

http://git.postgresql.org/?p=~davidfetter/postgresql/.git;a=commit;h=da63f9a82b9e902b5542f788b2e6e6bc95221793

> Not yet fixed:
>
> - detect certain queries those are not valid acroding to the standard
> - sort query names acording to the dependency

Is there something in the standard on how to do this? How to sort the
nodes other ways?

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Joshua D. Drake 2008-07-14 23:22:47 Re: Postgres-R source code release
Previous Message David Fetter 2008-07-14 23:02:28 Re: Exposing quals

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2008-07-14 23:11:55 Re: \d+ should display the storage options for columns
Previous Message Alvaro Herrera 2008-07-14 22:57:26 Re: \d+ should display the storage options for columns