Message style improvements

diff --git a/src/backend/access/common/toast_compression.c b/src/backend/access/common/toast_compression.c
index 9e9d4457aceaac47c5b97b71d5d57acf2a6f4603..845618349fe0a3a2f2c2dfbf28b838afa17f0bde 100644 (file)
--- a/src/backend/access/common/toast_compression.c
+++ b/src/backend/access/common/toast_compression.c
@@ -29,7 +29,7 @@ int           default_toast_compression = TOAST_PGLZ_COMPRESSION;
 #define NO_LZ4_SUPPORT() \
    ereport(ERROR, \
            (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), \
-            errmsg("unsupported LZ4 compression method"), \
+            errmsg("compression method lz4 not supported"), \
             errdetail("This functionality requires the server to be built with lz4 support."), \
             errhint("You need to rebuild PostgreSQL using %s.", "--with-lz4")))
 

diff --git a/src/backend/catalog/catalog.c b/src/backend/catalog/catalog.c
index 245d5363726555978a4e818d160f09560f47681f..7cabe741c66e07f4992e01de2f4b4916e20af416 100644 (file)
--- a/src/backend/catalog/catalog.c
+++ b/src/backend/catalog/catalog.c
@@ -375,10 +375,12 @@ GetNewOidWithIndex(Relation relation, Oid indexId, AttrNumber oidcolumn)
        if (retries >= retries_before_log)
        {
            ereport(LOG,
-                   (errmsg("still finding an unused OID within relation \"%s\"",
+                   (errmsg("still searching for an unused OID in relation \"%s\"",
                            RelationGetRelationName(relation)),
-                    errdetail("OID candidates were checked \"%llu\"  times, but no unused OID is yet found.",
-                              (unsigned long long) retries)));
+                    errdetail_plural("OID candidates have been checked %llu time, but no unused OID has been found yet.",
+                                     "OID candidates have been checked %llu times, but no unused OID has been found yet.",
+                                     retries,
+                                     (unsigned long long) retries)));
 
            /*
             * Double the number of retries to do before logging next until it
@@ -400,8 +402,10 @@ GetNewOidWithIndex(Relation relation, Oid indexId, AttrNumber oidcolumn)
    if (retries > GETNEWOID_LOG_THRESHOLD)
    {
        ereport(LOG,
-               (errmsg("new OID has been assigned in relation \"%s\" after \"%llu\" retries",
-                       RelationGetRelationName(relation), (unsigned long long) retries)));
+               (errmsg_plural("new OID has been assigned in relation \"%s\" after %llu retry",
+                              "new OID has been assigned in relation \"%s\" after %llu retries",
+                              retries,
+                              RelationGetRelationName(relation), (unsigned long long) retries)));
    }
 
    return newOid;

diff --git a/src/backend/catalog/pg_inherits.c b/src/backend/catalog/pg_inherits.c
index 1c37a438c391585e1d3d821b2e8a7aa8730cc156..ae990d487765abefbf24d100c2750b4b87b3c3df 100644 (file)
--- a/src/backend/catalog/pg_inherits.c
+++ b/src/backend/catalog/pg_inherits.c
@@ -593,7 +593,7 @@ DeleteInheritsTuple(Oid inhrelid, Oid inhparent, bool expect_detach_pending,
                         errmsg("cannot detach partition \"%s\"",
                                childname ? childname : "unknown relation"),
                         errdetail("The partition is being detached concurrently or has an unfinished detach."),
-                        errhint("Use ALTER TABLE ... DETACH PARTITION ... FINALIZE to complete the pending detach operation")));
+                        errhint("Use ALTER TABLE ... DETACH PARTITION ... FINALIZE to complete the pending detach operation.")));
            if (!detach_pending && expect_detach_pending)
                ereport(ERROR,
                        (errcode(ERRCODE_OBJECT_NOT_IN_PREREQUISITE_STATE),

diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 4e23c7fce5f4981a74d986f594e47936c69e1030..97a9725df75c8db0ddca00db362ce40ed0e5f458 100644 (file)
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -14773,7 +14773,7 @@ MarkInheritDetached(Relation child_rel, Relation parent_rel)
                           get_rel_name(inhForm->inhrelid),
                           get_namespace_name(parent_rel->rd_rel->relnamespace),
                           RelationGetRelationName(parent_rel)),
-                   errhint("Use ALTER TABLE ... DETACH PARTITION ... FINALIZE to complete the detach operation."));
+                   errhint("Use ALTER TABLE ... DETACH PARTITION ... FINALIZE to complete the pending detach operation."));
 
        if (inhForm->inhrelid == RelationGetRelid(child_rel))
        {

diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c
index b0d07c0e0bb700761bf148adac8022fe5e585c9d..ce8888cc3007406801fea569a59c8b503aa74106 100644 (file)
--- a/src/backend/postmaster/pgstat.c
+++ b/src/backend/postmaster/pgstat.c
@@ -1457,7 +1457,7 @@ pgstat_reset_shared_counters(const char *target)
        ereport(ERROR,
                (errcode(ERRCODE_INVALID_PARAMETER_VALUE),
                 errmsg("unrecognized reset target: \"%s\"", target),
-                errhint("Target must be \"archiver\", \"bgwriter\" or \"wal\".")));
+                errhint("Target must be \"archiver\", \"bgwriter\", or \"wal\".")));
 
    pgstat_setheader(&msg.m_hdr, PGSTAT_MTYPE_RESETSHAREDCOUNTER);
    pgstat_send(&msg, sizeof(msg));

diff --git a/src/backend/storage/file/fd.c b/src/backend/storage/file/fd.c
index e8cd7ef0886cf3366225f762aa8bb298f364bb2e..a340a5f6afe65fd224ac230f181480227bdb6dd5 100644 (file)
--- a/src/backend/storage/file/fd.c
+++ b/src/backend/storage/file/fd.c
@@ -3295,13 +3295,13 @@ do_syncfs(const char *path)
    {
        ereport(LOG,
                (errcode_for_file_access(),
-                errmsg("could not open %s: %m", path)));
+                errmsg("could not open file \"%s\": %m", path)));
        return;
    }
    if (syncfs(fd) < 0)
        ereport(LOG,
                (errcode_for_file_access(),
-                errmsg("could not sync filesystem for \"%s\": %m", path)));
+                errmsg("could not synchronize file system for file \"%s\": %m", path)));
    CloseTransientFile(fd);
 }
 #endif

diff --git a/src/backend/storage/ipc/standby.c b/src/backend/storage/ipc/standby.c
index 553b6e5460354688362d33c5c3fa1f4e8d81ca17..aeecaf6cabf5b84e5e191ef0231b71aef57c6964 100644 (file)
--- a/src/backend/storage/ipc/standby.c
+++ b/src/backend/storage/ipc/standby.c
@@ -303,7 +303,7 @@ LogRecoveryConflict(ProcSignalReason reason, TimestampTz wait_start,
    {
        ereport(LOG,
                errmsg("recovery still waiting after %ld.%03d ms: %s",
-                      msecs, usecs, _(get_recovery_conflict_desc(reason))),
+                      msecs, usecs, get_recovery_conflict_desc(reason)),
                nprocs > 0 ? errdetail_log_plural("Conflicting process: %s.",
                                                  "Conflicting processes: %s.",
                                                  nprocs, buf.data) : 0);
@@ -312,7 +312,7 @@ LogRecoveryConflict(ProcSignalReason reason, TimestampTz wait_start,
    {
        ereport(LOG,
                errmsg("recovery finished waiting after %ld.%03d ms: %s",
-                      msecs, usecs, _(get_recovery_conflict_desc(reason))));
+                      msecs, usecs, get_recovery_conflict_desc(reason)));
    }
 
    if (nprocs > 0)
@@ -1418,27 +1418,27 @@ LogStandbyInvalidations(int nmsgs, SharedInvalidationMessage *msgs,
 static const char *
 get_recovery_conflict_desc(ProcSignalReason reason)
 {
-   const char *reasonDesc = gettext_noop("unknown reason");
+   const char *reasonDesc = _("unknown reason");
 
    switch (reason)
    {
        case PROCSIG_RECOVERY_CONFLICT_BUFFERPIN:
-           reasonDesc = gettext_noop("recovery conflict on buffer pin");
+           reasonDesc = _("recovery conflict on buffer pin");
            break;
        case PROCSIG_RECOVERY_CONFLICT_LOCK:
-           reasonDesc = gettext_noop("recovery conflict on lock");
+           reasonDesc = _("recovery conflict on lock");
            break;
        case PROCSIG_RECOVERY_CONFLICT_TABLESPACE:
-           reasonDesc = gettext_noop("recovery conflict on tablespace");
+           reasonDesc = _("recovery conflict on tablespace");
            break;
        case PROCSIG_RECOVERY_CONFLICT_SNAPSHOT:
-           reasonDesc = gettext_noop("recovery conflict on snapshot");
+           reasonDesc = _("recovery conflict on snapshot");
            break;
        case PROCSIG_RECOVERY_CONFLICT_STARTUP_DEADLOCK:
-           reasonDesc = gettext_noop("recovery conflict on buffer deadlock");
+           reasonDesc = _("recovery conflict on buffer deadlock");
            break;
        case PROCSIG_RECOVERY_CONFLICT_DATABASE:
-           reasonDesc = gettext_noop("recovery conflict on database");
+           reasonDesc = _("recovery conflict on database");
            break;
        default:
            break;

diff --git a/src/backend/tcop/fastpath.c b/src/backend/tcop/fastpath.c
index 9fa8997cb30b7ab96cc60a9f07a9f243d262396a..6343dd269b45a5a1214c8220e0365ce4e048c09c 100644 (file)
--- a/src/backend/tcop/fastpath.c
+++ b/src/backend/tcop/fastpath.c
@@ -145,7 +145,7 @@ fetch_fp_info(Oid func_id, struct fp_info *fip)
    if (pp->prokind != PROKIND_FUNCTION || pp->proretset)
        ereport(ERROR,
                (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
-                errmsg("cannot call function %s via fastpath interface",
+                errmsg("cannot call function \"%s\" via fastpath interface",
                        NameStr(pp->proname))));
 
    /* watch out for catalog entries with more than FUNC_MAX_ARGS args */

diff --git a/src/backend/utils/adt/multirangetypes.c b/src/backend/utils/adt/multirangetypes.c
index fbcc27d0726c7190dfb46d9a03f6774aad7bf9eb..7aeec7617fc5ec9ab5a4ae217f969f5de3615abf 100644 (file)
--- a/src/backend/utils/adt/multirangetypes.c
+++ b/src/backend/utils/adt/multirangetypes.c
@@ -282,7 +282,7 @@ multirange_in(PG_FUNCTION_ARGS)
                (errcode(ERRCODE_INVALID_TEXT_REPRESENTATION),
                 errmsg("malformed multirange literal: \"%s\"",
                        input_str),
-                errdetail("Junk after right brace.")));
+                errdetail("Junk after closing right brace.")));
 
    ret = make_multirange(mltrngtypoid, rangetyp, range_count, ranges);
    PG_RETURN_MULTIRANGE_P(ret);
@@ -968,7 +968,7 @@ multirange_constructor2(PG_FUNCTION_ARGS)
    if (dims > 1)
        ereport(ERROR,
                (errcode(ERRCODE_CARDINALITY_VIOLATION),
-                errmsg("multiranges cannot be constructed from multi-dimensional arrays")));
+                errmsg("multiranges cannot be constructed from multidimensional arrays")));
 
    rngtypid = ARR_ELEMTYPE(rangeArray);
    if (rngtypid != rangetyp->type_id)

diff --git a/src/test/regress/expected/compression_1.out b/src/test/regress/expected/compression_1.out
index 15a23924ec7191887719209a860ad73af5e7caad..1ce2962d556a5a398d2ef357bff5f0aa1798df2f 100644 (file)
--- a/src/test/regress/expected/compression_1.out
+++ b/src/test/regress/expected/compression_1.out
@@ -14,7 +14,7 @@ Indexes:
     "idx" btree (f1)
 
 CREATE TABLE cmdata1(f1 TEXT COMPRESSION lz4);
-ERROR:  unsupported LZ4 compression method
+ERROR:  compression method lz4 not supported
 DETAIL:  This functionality requires the server to be built with lz4 support.
 HINT:  You need to rebuild PostgreSQL using --with-lz4.
 INSERT INTO cmdata1 VALUES(repeat('1234567890', 1004));
@@ -193,7 +193,7 @@ LINE 1: SELECT pg_column_compression(x) FROM compressmv;
                                              ^
 -- test compression with partition
 CREATE TABLE cmpart(f1 text COMPRESSION lz4) PARTITION BY HASH(f1);
-ERROR:  unsupported LZ4 compression method
+ERROR:  compression method lz4 not supported
 DETAIL:  This functionality requires the server to be built with lz4 support.
 HINT:  You need to rebuild PostgreSQL using --with-lz4.
 CREATE TABLE cmpart1 PARTITION OF cmpart FOR VALUES WITH (MODULUS 2, REMAINDER 0);
@@ -238,7 +238,7 @@ HINT:  Available values: pglz.
 SET default_toast_compression = 'pglz';
 -- test alter compression method
 ALTER TABLE cmdata ALTER COLUMN f1 SET COMPRESSION lz4;
-ERROR:  unsupported LZ4 compression method
+ERROR:  compression method lz4 not supported
 DETAIL:  This functionality requires the server to be built with lz4 support.
 HINT:  You need to rebuild PostgreSQL using --with-lz4.
 INSERT INTO cmdata VALUES (repeat('123456789', 4004));
@@ -272,7 +272,7 @@ ERROR:  relation "compressmv" does not exist
 ALTER TABLE cmpart1 ALTER COLUMN f1 SET COMPRESSION pglz;
 ERROR:  relation "cmpart1" does not exist
 ALTER TABLE cmpart2 ALTER COLUMN f1 SET COMPRESSION lz4;
-ERROR:  unsupported LZ4 compression method
+ERROR:  compression method lz4 not supported
 DETAIL:  This functionality requires the server to be built with lz4 support.
 HINT:  You need to rebuild PostgreSQL using --with-lz4.
 -- new data should be compressed with the current compression method
@@ -312,7 +312,7 @@ SELECT pg_column_compression(f1) FROM cmdata;
 -- test expression index
 DROP TABLE cmdata2;
 CREATE TABLE cmdata2 (f1 TEXT COMPRESSION pglz, f2 TEXT COMPRESSION lz4);
-ERROR:  unsupported LZ4 compression method
+ERROR:  compression method lz4 not supported
 DETAIL:  This functionality requires the server to be built with lz4 support.
 HINT:  You need to rebuild PostgreSQL using --with-lz4.
 CREATE UNIQUE INDEX idx1 ON cmdata2 ((f1 || f2));

diff --git a/src/test/regress/expected/multirangetypes.out b/src/test/regress/expected/multirangetypes.out
index 98ac592127b568328adb72b226eeca83667f37d3..3e941aec68c46289e430e6337c842d193b4eba1b 100644 (file)
--- a/src/test/regress/expected/multirangetypes.out
+++ b/src/test/regress/expected/multirangetypes.out
@@ -17,7 +17,7 @@ select '{(,)}.'::textmultirange;
 ERROR:  malformed multirange literal: "{(,)}."
 LINE 1: select '{(,)}.'::textmultirange;
                ^
-DETAIL:  Junk after right brace.
+DETAIL:  Junk after closing right brace.
 select '{[a,c),}'::textmultirange;
 ERROR:  malformed multirange literal: "{[a,c),}"
 LINE 1: select '{[a,c),}'::textmultirange;