summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHeikki Linnakangas2012-07-05 20:40:25 +0000
committerHeikki Linnakangas2012-07-05 20:40:25 +0000
commitde479e2ed28925ddd2907cad3e2d5dddd3b97199 (patch)
tree112b09832b0fae09ab3e9430877a5f7ad972d229
parentb66de4c6d7208d9ec420b912758377a3533c7a7d (diff)
Revert part of the previous patch that avoided using PLy_elog().
That caused the plpython_unicode regression test to fail on SQL_ASCII encoding, as evidenced by the buildfarm. The reason is that with the patch, you don't get the detail in the error message that you got before. That detail is actually very informative, so rather than just adjust the expected output, let's revert that part of the patch for now to make the buildfarm green again, and figure out some other way to avoid the recursion of PLy_elog() that doesn't lose the detail.
-rw-r--r--src/pl/plpython/plpy_util.c11
1 files changed, 1 insertions, 10 deletions
diff --git a/src/pl/plpython/plpy_util.c b/src/pl/plpython/plpy_util.c
index bf2953226f..4aabafc139 100644
--- a/src/pl/plpython/plpy_util.c
+++ b/src/pl/plpython/plpy_util.c
@@ -120,16 +120,7 @@ PLyUnicode_Bytes(PyObject *unicode)
rv = PyUnicode_AsEncodedString(unicode, serverenc, "strict");
if (rv == NULL)
- {
- /*
- * Use a plain ereport instead of PLy_elog to avoid recursion, if
- * the traceback formatting functions try to do unicode to bytes
- * conversion again.
- */
- ereport(ERROR,
- (errcode(ERRCODE_INTERNAL_ERROR),
- errmsg("could not convert Python Unicode object to PostgreSQL server encoding")));
- }
+ PLy_elog(ERROR, "could not convert Python Unicode object to PostgreSQL server encoding");
return rv;
}