aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlvaro Herrera <alvherre@alvh.no-ip.org>2019-04-30 10:00:38 -0400
committerAlvaro Herrera <alvherre@alvh.no-ip.org>2019-04-30 10:33:37 -0400
commit9f8b717a80c59e4a8e8091a8ff6c6f7666a69c33 (patch)
tree00a4494408f5579e258d719e37eaac59d13200ac
parent9a83afecb7ec6e828b7dbef3133f3dd8b0b4dddb (diff)
downloadpostgresql-9f8b717a80c59e4a8e8091a8ff6c6f7666a69c33.tar.gz
postgresql-9f8b717a80c59e4a8e8091a8ff6c6f7666a69c33.zip
Message style fixes
-rw-r--r--src/backend/access/table/tableamapi.c6
-rw-r--r--src/backend/access/transam/twophase.c4
-rw-r--r--src/backend/commands/indexcmds.c2
-rw-r--r--src/backend/commands/tablecmds.c6
-rw-r--r--src/backend/utils/misc/guc.c4
-rw-r--r--src/test/regress/output/tablespace.source2
6 files changed, 13 insertions, 11 deletions
diff --git a/src/backend/access/table/tableamapi.c b/src/backend/access/table/tableamapi.c
index bfd713f3af1..0053dc95cab 100644
--- a/src/backend/access/table/tableamapi.c
+++ b/src/backend/access/table/tableamapi.c
@@ -109,8 +109,8 @@ check_default_table_access_method(char **newval, void **extra, GucSource source)
if (strlen(*newval) >= NAMEDATALEN)
{
- GUC_check_errdetail("default_table_access_method is too long (maximum %d characters).",
- NAMEDATALEN - 1);
+ GUC_check_errdetail("%s is too long (maximum %d characters).",
+ "default_table_access_method", NAMEDATALEN - 1);
return false;
}
@@ -131,7 +131,7 @@ check_default_table_access_method(char **newval, void **extra, GucSource source)
{
ereport(NOTICE,
(errcode(ERRCODE_UNDEFINED_OBJECT),
- errmsg("Table access method \"%s\" does not exist",
+ errmsg("table access method \"%s\" does not exist",
*newval)));
}
else
diff --git a/src/backend/access/transam/twophase.c b/src/backend/access/transam/twophase.c
index a399c0052d9..21e028283c1 100644
--- a/src/backend/access/transam/twophase.c
+++ b/src/backend/access/transam/twophase.c
@@ -2190,12 +2190,12 @@ ProcessTwoPhaseBuffer(TransactionId xid,
if (fromdisk)
ereport(ERROR,
(errcode(ERRCODE_DATA_CORRUPTED),
- errmsg("corrupted two-phase state file for transaction \"%u\"",
+ errmsg("corrupted two-phase state file for transaction %u",
xid)));
else
ereport(ERROR,
(errcode(ERRCODE_DATA_CORRUPTED),
- errmsg("corrupted two-phase state in memory for transaction \"%u\"",
+ errmsg("corrupted two-phase state in memory for transaction %u",
xid)));
}
diff --git a/src/backend/commands/indexcmds.c b/src/backend/commands/indexcmds.c
index cb5d91e7640..fabba3bacd1 100644
--- a/src/backend/commands/indexcmds.c
+++ b/src/backend/commands/indexcmds.c
@@ -639,7 +639,7 @@ DefineIndex(Oid relationId,
if (partitioned && tablespaceId == MyDatabaseTableSpace)
ereport(ERROR,
(errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
- errmsg("cannot specify default tablespace for partitioned relation")));
+ errmsg("cannot specify default tablespace for partitioned relations")));
}
else
{
diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 2d0ef92badf..f6c9cf3b0ec 100644
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -7689,12 +7689,14 @@ ATAddForeignKeyConstraint(List **wqueue, AlteredTableInfo *tab, Relation rel,
fkconstraint->fk_upd_action == FKCONSTR_ACTION_CASCADE)
ereport(ERROR,
(errcode(ERRCODE_SYNTAX_ERROR),
- errmsg("invalid ON UPDATE action for foreign key constraint containing generated column")));
+ errmsg("invalid %s action for foreign key constraint containing generated column",
+ "ON UPDATE")));
if (fkconstraint->fk_del_action == FKCONSTR_ACTION_SETNULL ||
fkconstraint->fk_del_action == FKCONSTR_ACTION_SETDEFAULT)
ereport(ERROR,
(errcode(ERRCODE_SYNTAX_ERROR),
- errmsg("invalid ON DELETE action for foreign key constraint containing generated column")));
+ errmsg("invalid %s action for foreign key constraint containing generated column",
+ "ON DELETE")));
}
}
diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c
index f7f726b5aec..ed51da42341 100644
--- a/src/backend/utils/misc/guc.c
+++ b/src/backend/utils/misc/guc.c
@@ -11639,8 +11639,8 @@ check_recovery_target_name(char **newval, void **extra, GucSource source)
/* Use the value of newval directly */
if (strlen(*newval) >= MAXFNAMELEN)
{
- GUC_check_errdetail("recovery_target_name is too long (maximum %d characters).",
- MAXFNAMELEN - 1);
+ GUC_check_errdetail("%s is too long (maximum %d characters).",
+ "recovery_target_name", MAXFNAMELEN - 1);
return false;
}
return true;
diff --git a/src/test/regress/output/tablespace.source b/src/test/regress/output/tablespace.source
index 81e81594a79..7625374d466 100644
--- a/src/test/regress/output/tablespace.source
+++ b/src/test/regress/output/tablespace.source
@@ -106,7 +106,7 @@ Tablespace: "regress_tblspace"
CREATE TABLE testschema.dflt (a int PRIMARY KEY) PARTITION BY LIST (a) TABLESPACE pg_default;
ERROR: cannot specify default tablespace for partitioned relations
CREATE TABLE testschema.dflt (a int PRIMARY KEY USING INDEX TABLESPACE pg_default) PARTITION BY LIST (a);
-ERROR: cannot specify default tablespace for partitioned relation
+ERROR: cannot specify default tablespace for partitioned relations
SET default_tablespace TO 'pg_default';
CREATE TABLE testschema.dflt (a int PRIMARY KEY) PARTITION BY LIST (a) TABLESPACE regress_tblspace;
ERROR: cannot specify default tablespace for partitioned relations