diff options
author | Peter Eisentraut <peter@eisentraut.org> | 2019-02-11 10:31:36 +0100 |
---|---|---|
committer | Peter Eisentraut <peter@eisentraut.org> | 2019-02-11 10:31:36 +0100 |
commit | 12055c8f643afa5ed70d17acc7782c21299ffb69 (patch) | |
tree | 9a488d2498a2a2e368a6b5a3316d1a05575aaf6f | |
parent | eb68d71f99e945c89bbb1190086237be65ad784c (diff) | |
download | postgresql-12055c8f643afa5ed70d17acc7782c21299ffb69.tar.gz postgresql-12055c8f643afa5ed70d17acc7782c21299ffb69.zip |
Adjust error message
We usually don't use "namespace" in user-facing error messages. Also,
in master this was replaced by another error message referring to
"temporary objects", so we might as well use that here to avoid
introducing too many variants.
Discussion: https://www.postgresql.org/message-id/bbd3f8d9-e3d5-e5aa-4305-7f0121c3fa94@2ndquadrant.com
-rw-r--r-- | src/backend/access/transam/xact.c | 2 | ||||
-rw-r--r-- | src/test/modules/test_extensions/expected/test_extensions.out | 2 | ||||
-rw-r--r-- | src/test/regress/expected/temp.out | 8 |
3 files changed, 6 insertions, 6 deletions
diff --git a/src/backend/access/transam/xact.c b/src/backend/access/transam/xact.c index 2c517756c09..4cf5ffeedb4 100644 --- a/src/backend/access/transam/xact.c +++ b/src/backend/access/transam/xact.c @@ -2305,7 +2305,7 @@ PrepareTransaction(void) if ((MyXactFlags & XACT_FLAGS_ACCESSEDTEMPNAMESPACE)) ereport(ERROR, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), - errmsg("cannot PREPARE a transaction that has operated on temporary namespace"))); + errmsg("cannot PREPARE a transaction that has operated on temporary objects"))); /* * Likewise, don't allow PREPARE after pg_export_snapshot. This could be diff --git a/src/test/modules/test_extensions/expected/test_extensions.out b/src/test/modules/test_extensions/expected/test_extensions.out index 1eec5a37d33..b5cbdfcad4f 100644 --- a/src/test/modules/test_extensions/expected/test_extensions.out +++ b/src/test/modules/test_extensions/expected/test_extensions.out @@ -148,7 +148,7 @@ SELECT create_extension_with_temp_schema(); (1 row) PREPARE TRANSACTION 'twophase_extension'; -ERROR: cannot PREPARE a transaction that has operated on temporary namespace +ERROR: cannot PREPARE a transaction that has operated on temporary objects -- Clean up DROP TABLE test_ext4_tab; DROP FUNCTION create_extension_with_temp_schema(); diff --git a/src/test/regress/expected/temp.out b/src/test/regress/expected/temp.out index d6d8f25141e..97fe478caed 100644 --- a/src/test/regress/expected/temp.out +++ b/src/test/regress/expected/temp.out @@ -310,19 +310,19 @@ begin; create function pg_temp.twophase_func() returns void as $$ select '2pc_func'::text $$ language sql; prepare transaction 'twophase_func'; -ERROR: cannot PREPARE a transaction that has operated on temporary namespace +ERROR: cannot PREPARE a transaction that has operated on temporary objects -- Function drop create function pg_temp.twophase_func() returns void as $$ select '2pc_func'::text $$ language sql; begin; drop function pg_temp.twophase_func(); prepare transaction 'twophase_func'; -ERROR: cannot PREPARE a transaction that has operated on temporary namespace +ERROR: cannot PREPARE a transaction that has operated on temporary objects -- Operator creation begin; create operator pg_temp.@@ (leftarg = int4, rightarg = int4, procedure = int4mi); prepare transaction 'twophase_operator'; -ERROR: cannot PREPARE a transaction that has operated on temporary namespace +ERROR: cannot PREPARE a transaction that has operated on temporary objects -- These generate errors about temporary tables. begin; create type pg_temp.twophase_type as (a int); @@ -374,4 +374,4 @@ SELECT current_schema() ~ 'pg_temp' AS is_temp_schema; (1 row) PREPARE TRANSACTION 'twophase_search'; -ERROR: cannot PREPARE a transaction that has operated on temporary namespace +ERROR: cannot PREPARE a transaction that has operated on temporary objects |