aboutsummaryrefslogtreecommitdiff
path: root/src/backend/commands
diff options
context:
space:
mode:
authorPeter Eisentraut <peter@eisentraut.org>2022-09-24 18:38:35 -0400
committerPeter Eisentraut <peter@eisentraut.org>2022-09-24 18:38:35 -0400
commit517484b5820e9e20057ff066b5df7d09cbb5f464 (patch)
tree15a9d9f1fe55768e0d37c133eda70152ecee258e /src/backend/commands
parent8d985560fc1de6197323d4397d2a2bf762657463 (diff)
downloadpostgresql-517484b5820e9e20057ff066b5df7d09cbb5f464.tar.gz
postgresql-517484b5820e9e20057ff066b5df7d09cbb5f464.zip
Message style improvements
Diffstat (limited to 'src/backend/commands')
-rw-r--r--src/backend/commands/dbcommands.c2
-rw-r--r--src/backend/commands/publicationcmds.c4
-rw-r--r--src/backend/commands/subscriptioncmds.c6
-rw-r--r--src/backend/commands/trigger.c2
4 files changed, 6 insertions, 8 deletions
diff --git a/src/backend/commands/dbcommands.c b/src/backend/commands/dbcommands.c
index 5dfec5c6b05..034d14f39da 100644
--- a/src/backend/commands/dbcommands.c
+++ b/src/backend/commands/dbcommands.c
@@ -991,7 +991,7 @@ createdb(ParseState *pstate, const CreatedbStmt *stmt)
else
ereport(ERROR,
(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
- errmsg("invalid create database strategy %s", strategy),
+ errmsg("invalid create database strategy \"%s\"", strategy),
errhint("Valid strategies are \"wal_log\", and \"file_copy\".")));
}
diff --git a/src/backend/commands/publicationcmds.c b/src/backend/commands/publicationcmds.c
index aafb14c9a35..cfa7196f203 100644
--- a/src/backend/commands/publicationcmds.c
+++ b/src/backend/commands/publicationcmds.c
@@ -575,7 +575,7 @@ check_simple_rowfilter_expr_walker(Node *node, ParseState *pstate)
/* OK, supported */
break;
default:
- errdetail_msg = _("Expressions only allow columns, constants, built-in operators, built-in data types, built-in collations, and immutable built-in functions.");
+ errdetail_msg = _("Only columns, constants, built-in operators, built-in data types, built-in collations, and immutable built-in functions are allowed.");
break;
}
@@ -1359,7 +1359,7 @@ CheckAlterPublication(AlterPublicationStmt *stmt, HeapTuple tup,
(errcode(ERRCODE_OBJECT_NOT_IN_PREREQUISITE_STATE),
errmsg("publication \"%s\" is defined as FOR ALL TABLES",
NameStr(pubform->pubname)),
- errdetail("Tables from schema cannot be added to, dropped from, or set on FOR ALL TABLES publications.")));
+ errdetail("Schemas cannot be added to or dropped from FOR ALL TABLES publications.")));
/* Check that user is allowed to manipulate the publication tables. */
if (tables && pubform->puballtables)
diff --git a/src/backend/commands/subscriptioncmds.c b/src/backend/commands/subscriptioncmds.c
index ea42491ad99..ef4f9c175c2 100644
--- a/src/backend/commands/subscriptioncmds.c
+++ b/src/backend/commands/subscriptioncmds.c
@@ -430,10 +430,8 @@ check_publications(WalReceiverConn *wrconn, List *publications)
if (res->status != WALRCV_OK_TUPLES)
ereport(ERROR,
- errmsg_plural("could not receive publication from the publisher: %s",
- "could not receive list of publications from the publisher: %s",
- list_length(publications),
- res->err));
+ errmsg("could not receive list of publications from the publisher: %s",
+ res->err));
publicationsCopy = list_copy(publications);
diff --git a/src/backend/commands/trigger.c b/src/backend/commands/trigger.c
index 464e22517f0..ed0ab9ca8ee 100644
--- a/src/backend/commands/trigger.c
+++ b/src/backend/commands/trigger.c
@@ -1571,7 +1571,7 @@ renametrig(RenameStmt *stmt)
ereport(ERROR,
errmsg("cannot rename trigger \"%s\" on table \"%s\"",
stmt->subname, RelationGetRelationName(targetrel)),
- errhint("Rename trigger on partitioned table \"%s\" instead.",
+ errhint("Rename the trigger on the partitioned table \"%s\" instead.",
get_rel_name(get_partition_parent(relid, false))));