From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Make get_stack_depth_rlimit() handle RLIM_INFINITY more sanely. |
Date: | 2010-11-06 20:50:57 |
Message-ID: | E1PEpiz-0005jk-0v@gemulon.postgresql.org |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Make get_stack_depth_rlimit() handle RLIM_INFINITY more sanely.
Rather than considering this result as meaning "unknown", report LONG_MAX.
This won't change what superusers can set max_stack_depth to, but it will
cause InitializeGUCOptions() to set the built-in default to 2MB not 100kB.
The latter seems like a fairly unreasonable interpretation of "infinity".
Per my investigation of odd buildfarm results as well as an old complaint
from Heikki.
Since this should persuade all the buildfarm animals to use a reasonable
stack depth setting during "make check", revert previous patch that dumbed
down a recursive regression test to only 5 levels.
Branch
------
master
Details
-------
http://git.postgresql.org/gitweb?p=postgresql.git;a=commitdiff;h=dd1c781903811416db4e03383a4cb0bfc8cfac40
Modified Files
--------------
src/backend/tcop/postgres.c | 12 ++++++++----
src/backend/utils/misc/guc.c | 4 ++--
src/test/regress/expected/plpgsql.out | 2 +-
src/test/regress/sql/plpgsql.sql | 2 +-
4 files changed, 12 insertions(+), 8 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2010-11-06 23:37:14 | pgsql: Add support for detecting register-stack overrun on IA64. |
Previous Message | User Hinoue | 2010-11-06 05:48:21 | psqlodbc - psqlodbc: Fix a crash bug which was introduced in 9.0.0200. |