diff options
| author | Tom Lane | 2009-09-09 03:32:52 +0000 |
|---|---|---|
| committer | Tom Lane | 2009-09-09 03:32:52 +0000 |
| commit | 255f66efa9b648500a4f06b839d0256987406f0f (patch) | |
| tree | 985cab75c2b9cd5e911e7b5ea56dd1558b35aa40 /src/test | |
| parent | d69a419e682c2d39c2355105a7e5e2b90357c8f0 (diff) | |
Fix bug with WITH RECURSIVE immediately inside WITH RECURSIVE. 99% of the
code was already okay with this, but the hack that obtained the output
column types of a recursive union in advance of doing real parse analysis
of the recursive union forgot to handle the case where there was an inner
WITH clause available to the non-recursive term. Best fix seems to be to
refactor so that we don't need the "throwaway" parse analysis step at all.
Instead, teach the transformSetOperationStmt code to set up the CTE's output
column information after it's processed the non-recursive term normally.
Per report from David Fetter.
Diffstat (limited to 'src/test')
| -rw-r--r-- | src/test/regress/expected/with.out | 73 | ||||
| -rw-r--r-- | src/test/regress/sql/with.sql | 15 |
2 files changed, 88 insertions, 0 deletions
diff --git a/src/test/regress/expected/with.out b/src/test/regress/expected/with.out index 98003ebe6c6..a3e94e93d49 100644 --- a/src/test/regress/expected/with.out +++ b/src/test/regress/expected/with.out @@ -953,3 +953,76 @@ from int4_tbl; -2147483647 (5 rows) +-- +-- test for nested-recursive-WITH bug +-- +WITH RECURSIVE t(j) AS ( + WITH RECURSIVE s(i) AS ( + VALUES (1) + UNION ALL + SELECT i+1 FROM s WHERE i < 10 + ) + SELECT i FROM s + UNION ALL + SELECT j+1 FROM t WHERE j < 10 +) +SELECT * FROM t; + j +---- + 1 + 2 + 3 + 4 + 5 + 6 + 7 + 8 + 9 + 10 + 2 + 3 + 4 + 5 + 6 + 7 + 8 + 9 + 10 + 3 + 4 + 5 + 6 + 7 + 8 + 9 + 10 + 4 + 5 + 6 + 7 + 8 + 9 + 10 + 5 + 6 + 7 + 8 + 9 + 10 + 6 + 7 + 8 + 9 + 10 + 7 + 8 + 9 + 10 + 8 + 9 + 10 + 9 + 10 + 10 +(55 rows) + diff --git a/src/test/regress/sql/with.sql b/src/test/regress/sql/with.sql index 6eaa168d964..2cbaa42492f 100644 --- a/src/test/regress/sql/with.sql +++ b/src/test/regress/sql/with.sql @@ -485,3 +485,18 @@ from int4_tbl; select ( with cte(foo) as ( values(f1) ) values((select foo from cte)) ) from int4_tbl; + +-- +-- test for nested-recursive-WITH bug +-- +WITH RECURSIVE t(j) AS ( + WITH RECURSIVE s(i) AS ( + VALUES (1) + UNION ALL + SELECT i+1 FROM s WHERE i < 10 + ) + SELECT i FROM s + UNION ALL + SELECT j+1 FROM t WHERE j < 10 +) +SELECT * FROM t; |
