From: Bruce Momjian <bruce@momjian.us>
To: David G. Johnston <david.g.johnston@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Drew DeVault <sir@cmpwn.com>
Cc: Pg Docs <pgsql-docs@lists.postgresql.org>
Subject: Re: [PATCH] doc/queries.sgml: add missing comma
Date: Thu, 18 Aug 2022 14:10:48 -0400
Message-ID: <Yv6AqJuvnP3HelUP@momjian.us> (raw)
In-Reply-To: <CAKFQuwbp31e9bx38nfN16z0Vrqig7D=dh8WCN+aMBbxMaDw8Dg@mail.gmail.com>
References: <20211018091720.31299-1-sir@cmpwn.com>
<2007662.1634679184@sss.pgh.pa.us>
<CAKFQuwbp31e9bx38nfN16z0Vrqig7D=dh8WCN+aMBbxMaDw8Dg@mail.gmail.com>
On Tue, Oct 19, 2021 at 06:41:54PM -0700, David G. Johnston wrote:
> On Tue, Oct 19, 2021 at 2:33 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Drew DeVault <sir@cmpwn.com> writes:
> > Minor grammatical fix.
>
> Hmm, I'm not sure that reads any better than before.
>
> > <para>
> > - Strictly speaking, this process is iteration not recursion, but
> > + Strictly speaking, this process is iteration, not recursion, but
> > <literal>RECURSIVE</literal> is the terminology chosen by the SQL
> standards
>
> ... although I think this text is mine, so naturally I'd think
> that. Anyone else have an opinion?
>
>
>
> If I read that aloud to myself there is a comma after iteration.
>
> That said, given that a comma and a "but" later we use the word "RECURSIVE" the
> clarification that the process isn't recursion seems redundant. If one knows
> what it means to be "recursive" then they will understand the juxtaposition of
> "iteration" and "recursive" just fine. If they do not, I don't think adding
> the word "recursion" is going to make much difference.
>
> Thus:
> Strictly speaking, this process is iteration, but <literal>RECURSIVE</literal>
> is the terminology chosen by the SQL standards committee."
>
> Because the above sounds just fine, I'd argue that if one does leave "not
> recursion" it should be set off by a comma.
I went with new wording, which should make this even clearer; patch
attached.
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EDB https://enterprisedb.com
Indecision is a decision. Inaction is an action. Mark Batterson
Attachments:
[text/x-diff] recursion.diff (481B, ../Yv6AqJuvnP3HelUP@momjian.us/2-recursion.diff)
download | inline diff:diff --git a/doc/src/sgml/queries.sgml b/doc/src/sgml/queries.sgmlindex 516fbcbf37..51b752709e 100644--- a/doc/src/sgml/queries.sgml+++ b/doc/src/sgml/queries.sgml@@ -2172,7 +2172,7 @@ SELECT sum(n) FROM t;
<note>
<para>
- Strictly speaking, this process is iteration not recursion, but+ Strictly speaking, this process is iterative, not recursive, but
<literal>RECURSIVE</literal> is the terminology chosen by the SQL standards
committee.
</para>
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-docs@postgresql.org
Cc: bruce@momjian.us, david.g.johnston@gmail.com, tgl@sss.pgh.pa.us, sir@cmpwn.com, pgsql-docs@lists.postgresql.org
Subject: Re: [PATCH] doc/queries.sgml: add missing comma
In-Reply-To: <Yv6AqJuvnP3HelUP@momjian.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox