Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-17 Thread Tatsuo Ishii
> On Wed, Jul 16, 2008 at 01:57:04PM +0900, Tatsuo Ishii wrote: > > > > No idea. What do you think if we allow only one query name at the > > > > moment. > > > > > > I'm not sure I understand what that has to do with sorting. > > > > > > Please find attached a place where I've found some problem

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-15 Thread David Fetter
On Wed, Jul 16, 2008 at 01:57:04PM +0900, Tatsuo Ishii wrote: > > > No idea. What do you think if we allow only one query name at the > > > moment. > > > > I'm not sure I understand what that has to do with sorting. > > > > Please find attached a place where I've found some problems sorting by >

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-15 Thread Tatsuo Ishii
> On Wed, Jul 16, 2008 at 09:37:25AM +0900, Tatsuo Ishii wrote: > > > 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 cru

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-15 Thread David Fetter
On Wed, Jul 16, 2008 at 09:37:25AM +0900, Tatsuo Ishii wrote: > > 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 DISTIN

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-15 Thread Tatsuo Ishii
> 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 anot

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-14 Thread David Fetter
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 pl

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-09 Thread David Fetter
On Wed, Jul 09, 2008 at 04:43:27PM -0400, Aidan Van Dyk wrote: > * David Fetter <[EMAIL PROTECTED]> [080709 14:45]: > > On Tue, Jul 08, 2008 at 09:28:34PM -0400, Alvaro Herrera wrote: > > > > In fact, I fail to see the point of you providing the repo if > > > the upstream guys are apparently not

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-09 Thread Aidan Van Dyk
* David Fetter <[EMAIL PROTECTED]> [080709 14:45]: > On Tue, Jul 08, 2008 at 09:28:34PM -0400, Alvaro Herrera wrote: > > In fact, I fail to see the point of you providing the repo if the > > upstream guys are apparently not using it ... > > It's *very* early days to be dismissing git entirely.

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-09 Thread David Fetter
On Tue, Jul 08, 2008 at 09:28:34PM -0400, Alvaro Herrera wrote: > David Fetter wrote: > > 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). > > > > The git repository should now match this :) > > > > http://git.postgres

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-08 Thread Alvaro Herrera
David Fetter wrote: > 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). > > The git repository should now match this :) > > http://git.postgresql.org/?p=~davidfetter/postgresql/.git;a=summary > > Apparently, it's easiest

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-08 Thread David Fetter
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). The git repository should now match this :) http://git.postgresql.org/?p=~davidfetter/postgresql/.git;a=summary Apparently, it's easiest to clone via the following URL: ht

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-08 Thread Tatsuo Ishii
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 res

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-07 Thread David Fetter
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 < 50) as t > > test-#

Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP

2008-07-07 Thread Yoshiyuki Asaba
Hi, From: Hans-Juergen Schoenig <[EMAIL PROTECTED]> Subject: Re: [PATCHES] [HACKERS] WITH RECURSIVE updated to CVS TIP Date: Sat, 5 Jul 2008 10:43:57 +0200 > i did some quick testing with this wonderful patch. > it seems there are some flaws in there still: > > test=# ex