Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit0abc8fd

Browse files
committed
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 settingof 100kB and using ~ 10k per recursion level. While it might beinteresting to try to figure out why they're eating so much stack, it isn'tlikely that any fix for that would be back-patchable. So just change thetest to recurse only 5 levels. The extra levels don't prove anythingcorrectness-wise anyway.
1 parent70a0160 commit0abc8fd

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

‎src/test/regress/expected/plpgsql.out

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3996,8 +3996,8 @@ DROP FUNCTION nonsimple_expr_test();
39963996
create function recurse(float8) returns float8 as
39973997
$$
39983998
begin
3999-
if ($1< 10) then
4000-
return sql_recurse($1+ 1);
3999+
if ($1> 0) then
4000+
return sql_recurse($1- 1);
40014001
else
40024002
return $1;
40034003
end if;
@@ -4006,10 +4006,10 @@ $$ language plpgsql;
40064006
-- "limit" is to prevent this from being inlined
40074007
create function sql_recurse(float8) returns float8 as
40084008
$$ select recurse($1) limit 1; $$ language sql;
4009-
select recurse(0);
4009+
select recurse(5);
40104010
recurse
40114011
---------
4012-
10
4012+
0
40134013
(1 row)
40144014

40154015
create function error1(text) returns text language sql as

‎src/test/regress/sql/plpgsql.sql

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3199,8 +3199,8 @@ DROP FUNCTION nonsimple_expr_test();
31993199
createfunctionrecurse(float8) returns float8as
32003200
$$
32013201
begin
3202-
if ($1<10) then
3203-
return sql_recurse($1+1);
3202+
if ($1>0) then
3203+
return sql_recurse($1-1);
32043204
else
32053205
return $1;
32063206
end if;
@@ -3211,7 +3211,7 @@ $$ language plpgsql;
32113211
createfunctionsql_recurse(float8) returns float8as
32123212
$$select recurse($1)limit1; $$ language sql;
32133213

3214-
select recurse(0);
3214+
select recurse(5);
32153215

32163216
createfunctionerror1(text) returnstext language sqlas
32173217
$$SELECT relname::textFROM pg_class cWHEREc.oid= $1::regclass $$;

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp