Hint and detail punctuation
Hackorum builds and tests every patch posted to the lists, not only commitfest submissions. This is Hackorum's own CI rather than the PostgreSQL project's, and it is still under testing - please report anything that looks wrong.
You can run a PostgreSQL built from this patch straight from Docker, with no checkout and no build:
docker run --rm -p 5432:5432 ghcr.io/hackorum-dev/postgres-patch:t39771psql -h localhost -U postgresBuilt from patchset v3 (message #3), July 28, 2026 at 04:52 AM.
Every patchset is also pushed to a branch of our PostgreSQL fork, so you can check out the same tree CI built. Without a PostgreSQL checkout:
git clone --branch t39771_3 https://github.com/hackorum-dev/postgres.gitIn a checkout you already have, add the fork once:
git remote add hackorum https://github.com/hackorum-dev/postgres.gitthen, for this patchset and every later one:
git fetch hackorum t39771_3 && git checkout t39771_3Patchset v3 (message #3) is on t39771_3
While looking at error messages downstream, I noticed a few hints and details
in postgres which aren’t punctuated as per the style guide. The attached patch
fixes the ones where it seemed reasonable to end with a period.
cheers ./daniel
Attachments:
errhint_punctuation.patchapplication/octet-stream; name=errhint_punctuation.patch; x-unix-mode=0644Download+5-6
On Wed, Dec 05, 2018 at 05:22:25PM +0100, Daniel Gustafsson wrote:
While looking at error messages downstream, I noticed a few hints and details
in postgres which aren’t punctuated as per the style guide. The attached patch
fixes the ones where it seemed reasonable to end with a period.
Good point. I am spotting a couple more places:
src/backend/utils/misc/guc.c:
GUC_check_errdetail("effective_io_concurrency must be set to 0 on
platforms that lack posix_fadvise()");
src/backend/utils/misc/guc.c:
GUC_check_errdetail("recovery_target_timeline is not a valid number");
src/backend/utils/misc/guc.c:
GUC_check_errdetail("recovery_target_name is too long (maximum %d
characters)",
Thanks,
--
Michael
On 6 Dec 2018, at 05:46, Michael Paquier <michael@paquier.xyz> wrote:
On Wed, Dec 05, 2018 at 05:22:25PM +0100, Daniel Gustafsson wrote:
While looking at error messages downstream, I noticed a few hints and details
in postgres which aren’t punctuated as per the style guide. The attached patch
fixes the ones where it seemed reasonable to end with a period.Good point. I am spotting a couple more places:
src/backend/utils/misc/guc.c:
Ah, for some reason I hadn’t thought about looking at the GUCs. I agree with
your findings, patch updated.
cheers ./daniel
On Thu, Dec 06, 2018 at 09:42:26AM +0100, Daniel Gustafsson wrote:
Ah, for some reason I hadn’t thought about looking at the GUCs. I agree with
your findings, patch updated.
Thanks Daniel, that looks fine to me at quick glance. I'll try to get
that committed tomorrow my time if there are no objections until then.
--
Michael
On 2018-Dec-06, Daniel Gustafsson wrote:
On 6 Dec 2018, at 05:46, Michael Paquier <michael@paquier.xyz> wrote:
On Wed, Dec 05, 2018 at 05:22:25PM +0100, Daniel Gustafsson wrote:
While looking at error messages downstream, I noticed a few hints and details
in postgres which aren’t punctuated as per the style guide. The attached patch
fixes the ones where it seemed reasonable to end with a period.Good point. I am spotting a couple more places:
src/backend/utils/misc/guc.c:Ah, for some reason I hadn’t thought about looking at the GUCs. I agree with
your findings, patch updated.
LGTM.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
On Thu, Dec 06, 2018 at 09:44:22AM -0300, Alvaro Herrera wrote:
LGTM.
Thanks Álvaro for the additional lookup. I have committed the patch,
after updating the regression test output as this stuff was forgotten,
but that was easy enough to fix.
--
Michael
On 6 Dec 2018, at 23:51, Michael Paquier <michael@paquier.xyz> wrote:
On Thu, Dec 06, 2018 at 09:44:22AM -0300, Alvaro Herrera wrote:
LGTM.
Thanks Álvaro for the additional lookup. I have committed the patch,
after updating the regression test output as this stuff was forgotten,
but that was easy enough to fix.
Oh.. sorry about that, I should’ve known better. Thanks for tidying up!
cheers ./daniel
On Fri, Dec 07, 2018 at 01:27:52AM +0100, Daniel Gustafsson wrote:
Oh.. sorry about that, I should’ve known better. Thanks for tidying up!
No problem. Thanks for the report and for caring on the matter. So do
I.
--
Michael
On Wed, Dec 5, 2018 at 05:22:25PM +0100, Daniel Gustafsson wrote:
While looking at error messages downstream, I noticed a few hints and details
in postgres which aren’t punctuated as per the style guide. The attached patch
fixes the ones where it seemed reasonable to end with a period.
FYI, this was applied:
commit 730422afcdb6784bbe20efc65de72156d470b0c4
Author: Michael Paquier <michael@paquier.xyz>
Date: Fri Dec 7 07:47:42 2018 +0900
Fix some errhint and errdetail strings missing a period
As per the error message style guide of the documentation, those should
be full sentences.
Author: Daniel Gustafsson
Reviewed-by: Michael Paquier, Álvaro Herrera
Discussion: https://1E8D49B4-16BC-4420-B4ED-58501D9E076B@yesql.se
--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +