aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNeil Conway <neilc@samurai.com>2006-02-28 20:56:14 +0000
committerNeil Conway <neilc@samurai.com>2006-02-28 20:56:14 +0000
commit2b8afe6193c6e64d956c400397c421be4065acaa (patch)
tree2aa7c21efd3aae3252ec65679f6deaca1fb6bf38
parent87daae1143520309da299b78a8e7a68f141a268f (diff)
downloadpostgresql-2b8afe6193c6e64d956c400397c421be4065acaa.tar.gz
postgresql-2b8afe6193c6e64d956c400397c421be4065acaa.zip
Tweak the error message emitted when a void-returning PL/Python function
does not return None, per suggestion from Tom.
-rw-r--r--src/pl/plpython/expected/plpython_test.out4
-rw-r--r--src/pl/plpython/plpython.c6
2 files changed, 5 insertions, 5 deletions
diff --git a/src/pl/plpython/expected/plpython_test.out b/src/pl/plpython/expected/plpython_test.out
index 92243738f66..6aa550dad5d 100644
--- a/src/pl/plpython/expected/plpython_test.out
+++ b/src/pl/plpython/expected/plpython_test.out
@@ -190,8 +190,8 @@ SELECT test_void_func1(), test_void_func1() IS NULL AS "is null";
(1 row)
SELECT test_void_func2(); -- should fail
-ERROR: unexpected return value from plpython procedure
-DETAIL: void-returning functions must return "None"
+ERROR: invalid return value from plpython function
+DETAIL: Functions returning type "void" must return "None".
SELECT test_return_none(), test_return_none() IS NULL AS "is null";
test_return_none | is null
------------------+---------
diff --git a/src/pl/plpython/plpython.c b/src/pl/plpython/plpython.c
index 14c805a29ef..cea843f343a 100644
--- a/src/pl/plpython/plpython.c
+++ b/src/pl/plpython/plpython.c
@@ -29,7 +29,7 @@
* MAINTENANCE, SUPPORT, UPDATES, ENHANCEMENTS, OR MODIFICATIONS.
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/pl/plpython/plpython.c,v 1.72 2006/02/28 20:03:52 neilc Exp $
+ * $PostgreSQL: pgsql/src/pl/plpython/plpython.c,v 1.73 2006/02/28 20:56:14 neilc Exp $
*
*********************************************************************
*/
@@ -769,8 +769,8 @@ PLy_function_handler(FunctionCallInfo fcinfo, PLyProcedure * proc)
if (plrv != Py_None)
ereport(ERROR,
(errcode(ERRCODE_DATATYPE_MISMATCH),
- errmsg("unexpected return value from plpython procedure"),
- errdetail("void-returning functions must return \"None\"")));
+ errmsg("invalid return value from plpython function"),
+ errdetail("Functions returning type \"void\" must return \"None\".")));
fcinfo->isnull = false;
rv = (Datum) 0;