postgres.git / summary / log / commit / refs

commit    c4a2bd1fa9a08237a835d106817e33bd1ef77176
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Wed Nov 03 17:42:19 2010 +0000
First released in: 9.1.0, 8.1.23, 8.2.19, 8.3.13, 8.4.6, 9.0.2

    Reduce recursion depth in recently-added regression test.
    
    Some buildfarm members fail the test with the original depth of 10 levels,
    apparently because they are running at the minimum max_stack_depth setting
    of 100kB and using ~ 10k per recursion level.  While it might be
    interesting to try to figure out why they're eating so much stack, it isn't
    likely that any fix for that would be back-patchable.  So just change the
    test to recurse only 5 levels.  The extra levels don't prove anything
    correctness-wise anyway.


 src/test/regress/expected/plpgsql.out | 8 ++++----
 src/test/regress/sql/plpgsql.sql      | 6 +++---
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/test/regress/expected/plpgsql.out b/src/test/regress/expected/plpgsql.out
index ff1a9604df7..4a93ee24148 100644
--- a/src/test/regress/expected/plpgsql.out
+++ b/src/test/regress/expected/plpgsql.out
@@ -2985,8 +2985,8 @@ DROP FUNCTION nonsimple_expr_test();
 create function recurse(float8) returns float8 as
 $$
 begin
-  if ($1 < 10) then
-    return sql_recurse($1 + 1);
+  if ($1 > 0) then
+    return sql_recurse($1 - 1);
   else
     return $1;
   end if;
@@ -2995,10 +2995,10 @@ $$ language plpgsql;
 -- "limit" is to prevent this from being inlined
 create function sql_recurse(float8) returns float8 as
 $$ select recurse($1) limit 1; $$ language sql;
-select recurse(0);
+select recurse(5);
  recurse 
 ---------
-      10
+       0
 (1 row)
 
 create function error1(text) returns text language sql as
diff --git a/src/test/regress/sql/plpgsql.sql b/src/test/regress/sql/plpgsql.sql
index 37bdb72fb7e..ea7aa129f15 100644
--- a/src/test/regress/sql/plpgsql.sql
+++ b/src/test/regress/sql/plpgsql.sql
@@ -2490,8 +2490,8 @@ DROP FUNCTION nonsimple_expr_test();
 create function recurse(float8) returns float8 as
 $$
 begin
-  if ($1 < 10) then
-    return sql_recurse($1 + 1);
+  if ($1 > 0) then
+    return sql_recurse($1 - 1);
   else
     return $1;
   end if;
@@ -2502,7 +2502,7 @@ $$ language plpgsql;
 create function sql_recurse(float8) returns float8 as
 $$ select recurse($1) limit 1; $$ language sql;
 
-select recurse(0);
+select recurse(5);
 
 create function error1(text) returns text language sql as
 $$ SELECT relname::text FROM pg_class c WHERE c.oid = $1::regclass $$;


[parent: 6b1952b07176]