]> granicus.if.org Git - postgresql/commitdiff
Reduce recursion depth in recently-added regression test.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 3 Nov 2010 17:42:24 +0000 (13:42 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 3 Nov 2010 17:42:24 +0000 (13:42 -0400)
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
src/test/regress/sql/plpgsql.sql

index bfa47eb3ca936fab3886aa8c549a37d37f1a948f..3fdf71cc21ca67940a5e4dbf2b33665cc0386c07 100644 (file)
@@ -2790,8 +2790,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;
@@ -2800,10 +2800,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
index 11227ec5b73271a27b7892b761357e479c2a98b7..9a602c357536144493ddf010ec9b66e8967c6846 100644 (file)
@@ -2345,8 +2345,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;
@@ -2357,7 +2357,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 $$;