Re: Change JOIN tutorial to focus more on explicit joins

From: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>
To: Jürgen Purtz <juergen(at)purtz(dot)de>
Cc: Anastasia Lubennikova <a(dot)lubennikova(at)postgrespro(dot)ru>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Change JOIN tutorial to focus more on explicit joins
Date: 2020-11-30 20:25:58
Message-ID: CAKFQuwauhGRnwqGamn6oHrdP-3v2SR0ELhe_a3w0BtjpkV5q_g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs pgsql-hackers

On Mon, Nov 30, 2020 at 1:15 PM Jürgen Purtz <juergen(at)purtz(dot)de> wrote:

> On 30.11.20 20:45, Anastasia Lubennikova wrote:
> > As far as I see something got committed and now the discussion is stuck
> in arguing about parenthesis.
> > FWIW, I think it is a matter of personal taste. Maybe we can compromise
> on simply leaving this part unchanged.
>
> With or without parenthesis is a little more than a personal taste, but
> it's a very tiny detail. I'm happy with either of the two variants.
>
>
Sorry, I managed to overlook the most recent patch.

I admitted my use of parentheses was incorrect and I don't see anyone else
defending them. Please remove them.

Minor typos:

"the database compare" -> needs an "s" (compares)

"In this case, the definition how to compare their rows." -> remove,
redundant with the first sentence

"The results from the older implicit syntax, and the newer explicit JOIN/ON
syntax, are identical" -> move the commas around to what is shown here

David J.

In response to

Responses

Browse pgsql-docs by date

  From Date Subject
Next Message Shinya11.Kato 2020-12-01 04:26:46 RE: index items for pg_stat_progress_xxx views
Previous Message Jürgen Purtz 2020-11-30 20:15:11 Re: Change JOIN tutorial to focus more on explicit joins

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Korotkov 2020-11-30 20:39:24 Re: range_agg
Previous Message Anastasia Lubennikova 2020-11-30 20:21:19 Re: [patch] [doc] Introduce view updating options more succinctly