INSERT documentation

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
nat
Reply | Threaded
Open this post in threaded view
|

INSERT documentation

nat
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 "ON CONFLICT Clause" section of the official INSERT documentation (
https://www.postgresql.org/docs/10/static/sql-insert.html#sql-on-conflict )
you can read:
"index_column_name
The name of a table_name column."
This seems misleading to me as multiple columns may be named there.  The
second sentence of this section's second paragraph is clear ("When
performing inference, it consists of one or more index_column_name") 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/

Thank you for the useful and appreciated work on PostgreSQL.

--
Sent via pgsql-docs mailing list ([hidden email])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs
Reply | Threaded
Open this post in threaded view
|

Re: INSERT documentation

Marko Tiikkaja-4
On Wed, Oct 11, 2017 at 5:34 PM, <[hidden email]> 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
Previous Thread Next Thread