diff options
author | Peter Eisentraut | 2022-09-24 22:38:35 +0000 |
---|---|---|
committer | Peter Eisentraut | 2022-09-24 22:41:25 +0000 |
commit | 26f7802beb2a4aafa0903f5bedeb7f1fa6f4f358 (patch) | |
tree | dc9a4b176f6a37ca32b2667deb5e635196a0d70f /src/backend/commands | |
parent | a6bc3301925e1a8ad1f58da629b9dd55bc4b8d9c (diff) |
Message style improvements
Diffstat (limited to 'src/backend/commands')
-rw-r--r-- | src/backend/commands/dbcommands.c | 2 | ||||
-rw-r--r-- | src/backend/commands/publicationcmds.c | 4 | ||||
-rw-r--r-- | src/backend/commands/subscriptioncmds.c | 6 | ||||
-rw-r--r-- | src/backend/commands/trigger.c | 2 |
4 files changed, 6 insertions, 8 deletions
diff --git a/src/backend/commands/dbcommands.c b/src/backend/commands/dbcommands.c index e0753c1badc..96b46cbc020 100644 --- a/src/backend/commands/dbcommands.c +++ b/src/backend/commands/dbcommands.c @@ -992,7 +992,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 7ffad096aa1..99011eec9fa 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 d042abe3414..f3bfcca434c 100644 --- a/src/backend/commands/subscriptioncmds.c +++ b/src/backend/commands/subscriptioncmds.c @@ -461,10 +461,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 6f5a5262c73..0ec8d84a1b5 100644 --- a/src/backend/commands/trigger.c +++ b/src/backend/commands/trigger.c @@ -1532,7 +1532,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)))); |