(2014/04/15 15:27), Etsuro Fujita wrote:
(2014/04/14 23:53), Robert Haas wrote:
On Fri, Apr 11, 2014 at 5:00 AM, Etsuro Fujita
<fujita.ets...@lab.ntt.co.jp> wrote:
Attached is an updated version of the patch.
I think the other changes deserve to be considered separately, and in
particular I'm still not sure it's a good idea to document both OF
type_name and type_name.
I've agreed on that point, but I think apart from the others, the
trivial typo should be corrected. Patch attached
(doc-altertable-typo.patch).
I noticed the description of index_name should also be corrected,
because it is currently used not only in CLUSTER ON, but in ADD
table_constraint_using_index. (It will also be used in REPLICA IDENTITY
in 9.4.) Patch attached (doc-altertable-indexname.patch).
Thanks,
Best regards,
Etsuro Fujita
diff --git a/doc/src/sgml/ref/alter_table.sgml
b/doc/src/sgml/ref/alter_table.sgml
index 85705e9..42167d8 100644
--- a/doc/src/sgml/ref/alter_table.sgml
+++ b/doc/src/sgml/ref/alter_table.sgml
@@ -720,7 +720,7 @@ ALTER TABLE [ IF EXISTS ] <replaceable
class="PARAMETER">name</replaceable>
</varlistentry>
<varlistentry>
- <term><replaceable class="PARAMETER">type</replaceable></term>
+ <term><replaceable class="PARAMETER">data_type</replaceable></term>
<listitem>
<para>
Data type of the new column, or new data type for an existing
diff --git a/doc/src/sgml/ref/alter_table.sgml
b/doc/src/sgml/ref/alter_table.sgml
index 85705e9..0354dde 100644
--- a/doc/src/sgml/ref/alter_table.sgml
+++ b/doc/src/sgml/ref/alter_table.sgml
@@ -805,7 +805,7 @@ ALTER TABLE [ IF EXISTS ] <replaceable
class="PARAMETER">name</replaceable>
<term><replaceable class="PARAMETER">index_name</replaceable></term>
<listitem>
<para>
- The index name on which the table should be marked for clustering.
+ The name of an existing index.
</para>
</listitem>
</varlistentry>
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
