| From: | Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> |
|---|---|
| To: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | A few message wording/formatting cleanup patches |
| Date: | 2026-05-28 03:16:22 |
| Message-ID: | 20260528.121622.1662808269492494574.horikyota.ntt@gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello,
While translating error messages into Japanese, I came across some
unusually formatted messages and messages that are difficult to
translate naturally into Japanese.
0001: 0001-Make-propgraph-object-descriptions-translatable.patch
getObjectDescription() currently builds the object name for
PropgraphElementRelationId incrementally with appendStringInfo(), but
that works poorly with Japanese because it effectively fixes the word
order. We probably need to construct the whole name from a single
format string instead.
0002: 0002-Use-double-quotes-in-message.patch
fe-protocol3.c has the following message containing backquotes:
> libpq_append_conn_error(conn,
> "server did not report the unsupported `_pq_.test_protocol_negotiation` parameter in its protocol negotiation message");
but that doesn't seem to match our usual style. Nearby messages use
double quotes instead.
0003: 0003-Add-missing-period-to-HINT-message.patch
In be-secure-openssl.c, the following HINT message is missing a
trailing period:
> errhint("If ssl_sni is enabled then add configuration to "%s", else "%s"",
0004: 0004-Use-singular-datachecksum-consistently-in-process-na.patch
In datachecksum_state.c, the launcher process is referred to in two
different ways: "datachecksum launcher" and "datachecksums
launcher". Considering the worker process name, I think the former is
probably the intended one, so this patch makes the naming consistent
accordingly.
That said, I can also imagine an interpretation where "datachecksums"
was chosen intentionally to refer to the checksum feature or checksum
set as a whole, so I'm not entirely sure whether this should be
considered a real issue or just a stylistic inconsistency.
Still, having both forms coexist seems somewhat error-prone in
practice, especially when typing or searching symbol names.
Regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
| Attachment | Content-Type | Size |
|---|---|---|
| 0001-Make-propgraph-object-descriptions-translatable.patch | text/x-patch | 6.4 KB |
| 0002-Use-double-quotes-in-message.patch | text/x-patch | 1.1 KB |
| 0003-Add-missing-period-to-HINT-message.patch | text/x-patch | 1006 bytes |
| 0004-Use-singular-datachecksum-consistently-in-process-na.patch | text/x-patch | 37.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | jian he | 2026-05-28 03:23:45 | Re: FOR PORTION OF does not recompute GENERATED STORED columns that depend on the range column |
| Previous Message | Nisha Moond | 2026-05-28 03:12:03 | Re: pg_createsubscriber: Fix incorrect handling of cleanup flags |