Incorrect grammar

Started by PG Bug reporting form2 months ago2 messagesdocs
Beta feature

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.

appliessuccessCI history

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:t253119
psql -h localhost -U postgres

Built from patchset v2 (message #2), September 20, 2026 at 06:19 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 t253119_2 https://github.com/hackorum-dev/postgres.git

In a checkout you already have, add the fork once:

git remote add hackorum https://github.com/hackorum-dev/postgres.git

then, for this patchset and every later one:

git fetch hackorum t253119_2 && git checkout t253119_2

Patchset v2 (message #2) is on t253119_2

Jump to latest
#1PG Bug reporting form
noreply@postgresql.org

The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/18/tutorial-arch.html
Description:

The collection of words "The user's client (frontend) application that wants
to perform database operations." is not a complete sentence. (This is from
https://www.postgresql.org/docs/current/tutorial-arch.html). Perhaps you
want to say "A client process is an application that wants to perform
database operations" or similar.

#2Laurenz Albe
laurenz.albe@cybertec.at
In reply to: PG Bug reporting form (#1)
Re: Incorrect grammar

On Sat, 2026-07-18 at 00:13 +0000, PG Doc comments form wrote:

The collection of words "The user's client (frontend) application that wants
to perform database operations." is not a complete sentence. (This is from
https://www.postgresql.org/docs/current/tutorial-arch.html). Perhaps you
want to say "A client process is an application that wants to perform
database operations" or similar.

It makes sense if you look at the sentence before the enumeration:

A PostgreSQL session consists of the following cooperating processes (programs):

- A server process, which [...]. The database server program is called postgres.

- The user's client (frontend) application that wants to perform database operations.

Perhaps the source of the problem is that the fragment starts with a capitalized
word and ends with a period. But since there are sentences following the fragment,
I am not sure how the correct punctuation would be.

Perhaps the fragments could be turned into whole sentences, like

Second, there is the user's client (frontend) application that wants to perform
database operations.

Would the attached patch improve the situation for you?

Yours,
Laurenz Albe

Attachments:

t253119_2
v1-0001-Use-whole-sentences-in-the-tutorial.patchtext/x-patch; charset=UTF-8; name=v1-0001-Use-whole-sentences-in-the-tutorial.patchDownload+2-3