Re: Add missing period to DETAIL messages - Mailing list pgsql-hackers

On Mon, 13 Apr 2026 at 06:36, Peter Smith <smithpb2250@gmail.com> wrote:
>
> (here's the missing attachment)
>
> On Mon, Apr 13, 2026 at 11:04 AM Peter Smith <smithpb2250@gmail.com> wrote:
> >
> > Found one more oversight.
> >
> > PSA v3.

Should these also be updated:
1) ginfuncs.c
if (opaq->flags != (GIN_DATA | GIN_LEAF | GIN_COMPRESSED))
ereport(ERROR,
(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
errmsg("input page is not a compressed GIN data leaf page"),
errdetail("Flags %04X, expected %04X",
   opaq->flags,
   (GIN_DATA | GIN_LEAF | GIN_COMPRESSED))));

2) shell_archive.c
ereport(lev,
(errmsg("archive command failed with exit code %d",
WEXITSTATUS(rc)),
errdetail("The failed archive command was: %s",
   xlogarchcmd)));

3) shell_archive.c
ereport(lev,
(errmsg("archive command was terminated by exception 0x%X",
WTERMSIG(rc)),
errhint("See C include file \"ntstatus.h\" for a description of the
hexadecimal value."),
errdetail("The failed archive command was: %s",
   xlogarchcmd)));

4) shell_archive.c
ereport(lev,
(errmsg("archive command was terminated by signal %d: %s",
WTERMSIG(rc), pg_strsignal(WTERMSIG(rc))),
errdetail("The failed archive command was: %s",
   xlogarchcmd)));

5) shell_archive.c
ereport(lev,
(errmsg("archive command exited with unrecognized status %d",
rc),
errdetail("The failed archive command was: %s",
   xlogarchcmd)));

6) matview.c
ereport(ERROR,
(errcode(ERRCODE_CARDINALITY_VIOLATION),
errmsg("new data for materialized view \"%s\" contains duplicate rows
without any null columns",
RelationGetRelationName(matviewRel)),
errdetail("Row: %s",
   SPI_getvalue(SPI_tuptable->vals[0], SPI_tuptable->tupdesc, 1))));

Regards,
Vignesh