pg.ddx.io pgsql-docs@postgresql.org mailing list archive
help / color / mirror / Atom feedALTER TABLE documentation nitpick
2+ messages / 2 participants
[nested] [flat]
* ALTER TABLE documentation nitpick
@ 2015-04-10 02:20 Amit Langote <amitlangote09@gmail.com>
0 siblings, 1 reply; 2+ messages in thread
From: Amit Langote @ 2015-04-10 02:20 UTC (permalink / raw)
To: pgsql-docs
Hi,
ALTER TABLE [ IF EXISTS ] [ ONLY ] name [ * ]
action [, ... ]
Why the description of [ IF EXISTS ] part above is listed with that of
actions; it seems to give an impression that it is one of the ALTER
TABLE actions. I think it should be one of the lines in the paragraph
just above where the description of actions begins.
I propose to change is it like attached.
Thoughts?
Thanks,
Amit
--
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs
Attachments:
alter-table-doc-fix.patch (application/octet-stream, 1.3K) download
diff --git a/doc/src/sgml/ref/alter_table.sgml b/doc/src/sgml/ref/alter_table.sgml
index 6a82730..0a86b46 100644
--- a/doc/src/sgml/ref/alter_table.sgml
+++ b/doc/src/sgml/ref/alter_table.sgml
@@ -92,7 +92,9 @@ ALTER TABLE ALL IN TABLESPACE <replaceable class="PARAMETER">name</replaceable>
There are several subforms described below. Note that the lock level required
may differ for each subform. An <literal>ACCESS EXCLUSIVE</literal> lock is held
unless explicitly noted. When multiple subcommands are listed, the lock
- held will be the strictest one required from any subcommand.
+ held will be the strictest one required from any subcommand. If <literal>IF EXISTS
+ </literal> is specified and the table does not exist, no error is thrown. In this
+ case a notice is issued instead.
<variablelist>
<varlistentry>
@@ -122,16 +124,6 @@ ALTER TABLE ALL IN TABLESPACE <replaceable class="PARAMETER">name</replaceable>
</varlistentry>
<varlistentry>
- <term><literal>IF EXISTS</literal></term>
- <listitem>
- <para>
- Do not throw an error if the table does not exist. A notice is issued
- in this case.
- </para>
- </listitem>
- </varlistentry>
-
- <varlistentry>
<term><literal>SET DATA TYPE</literal></term>
<listitem>
<para>
* Re: ALTER TABLE documentation nitpick
@ 2015-04-24 17:24 Peter Eisentraut <peter_e@gmx.net>
parent: Amit Langote <amitlangote09@gmail.com>
0 siblings, 0 replies; 2+ messages in thread
From: Peter Eisentraut @ 2015-04-24 17:24 UTC (permalink / raw)
To: Amit Langote <amitlangote09@gmail.com>; pgsql-docs
On 4/9/15 10:20 PM, Amit Langote wrote:
> Hi,
>
> ALTER TABLE [ IF EXISTS ] [ ONLY ] name [ * ]
> action [, ... ]
>
> Why the description of [ IF EXISTS ] part above is listed with that of
> actions; it seems to give an impression that it is one of the ALTER
> TABLE actions. I think it should be one of the lines in the paragraph
> just above where the description of actions begins.
If moved it to the Parameters section, which is where it is on other
reference pages.
--
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs
^ permalink raw reply [nested|flat] 2+ messages in thread
end of thread, other threads:[~2015-04-24 17:24 UTC | newest]
Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2015-04-10 02:20 ALTER TABLE documentation nitpick Amit Langote <amitlangote09@gmail.com>
2015-04-24 17:24 ` Peter Eisentraut <peter_e@gmx.net>
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