public inbox for [email protected]  

help / color / mirror / Atom feed

From: Gavin Flower <[email protected]>
To: Alvaro Herrera <[email protected]>
To: [email protected]
Subject: Re: Adding a Column documentation is misleading
Date: Fri, 8 Nov 2019 09:36:13 +1300
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>

On 07/11/2019 02:13, Alvaro Herrera wrote:
> On 2019-Nov-06, PG Doc comments form wrote:
>
>> Page: https://www.postgresql.org/docs/12/ddl-alter.html
>> Description:
>>
>> In 5.6.1. Adding a Column, there is a kind of example 'ALTER TABLE products
>> ADD COLUMN description text;'
>>
>> The words 'description' and 'text' are misleading -- as according to the
>> formal documentation of the SQL command
>> (https://www.postgresql.org/docs/12/sql-altertable.html), they should be
>> 'column_name' and 'data_type'.
> Well, it's an example, so "description" is the column name and "text" is
> its data type.  If you had a table called products, you could run that
> command and it would work just fine (assuming you don't already have a
> column called description, doh).
>
> Maybe the example could be made clearer by using some other column name
> and some other data type, so that they don't resemble english prose or
> keywords.  Maybe "alter table cities add column year_founded integer".
> Do you want to propose something better than that?
>
>> A similar problem exists for removing a column, and other actions.
> Let's hear your proposed changes.
>
Now it's explained, it is obvious!

Sorry, for the noise.


Cheers,
Gavin

view thread (4+ messages)

reply

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: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Adding a Column documentation is misleading
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox