Re: INSERT documentation

From: Marko Tiikkaja <marko(at)joh(dot)to>
To: nat(at)makarevitch(dot)org
Cc: pgsql-docs <pgsql-docs(at)postgresql(dot)org>
Subject: Re: INSERT documentation
Date: 2017-10-12 12:54:49
Message-ID: CAL9smLCSvja9czoPtekezq-s4ugeG6DmWkMfU4FLM4Hf13xz0w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs

On Wed, Oct 11, 2017 at 5:34 PM, <nat(at)makarevitch(dot)org> wrote:

> The following documentation comment has been logged on the website:
>
> Page: https://www.postgresql.org/docs/10/static/sql-insert.html
> Description:
>
> Hi,
>
> In the &quot;ON CONFLICT Clause&quot; section of the official INSERT
> documentation (
> https://www.postgresql.org/docs/10/static/sql-insert.html#sql-on-conflict
> )
> you can read:
> &quot;index_column_name
> The name of a table_name column.&quot;
> This seems misleading to me as multiple columns may be named there. The
> second sentence of this section&#39;s second paragraph is clear (&quot;When
> performing inference, it consists of one or more index_column_name&quot;)
> but a
> reader jumping to the detail may miss this information.
>
> Suggestion: s/The name of a table_name column/The name of at least one
> column/
>

But index_column_name *is*the name of only a single column. The syntax is
(irrelevant parts omitted):

( index_column_name [, ...] )

which means "opening parenthesis, index_column_name followed by zero or
more index_column_name, and a closing parenthesis". The documentation
seems correct to me.

.m

In response to

Browse pgsql-docs by date

  From Date Subject
Next Message Tom Lane 2017-10-12 15:37:08 Re: version 10 release note mistype in release-10.sgml
Previous Message bunyaminsevgin 2017-10-12 11:46:43 Full Stack Developer