diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2012-09-22 18:35:22 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2012-09-22 18:35:22 -0400 |
| commit | 31510194cc9d87b355cb56e7d88c18c985d7a32a (patch) | |
| tree | bbfc0569d0e8d881d3fba8d4532257646c4b3943 /doc/src | |
| parent | fcc1576687a122059db311791e45e2c5edc5d89b (diff) | |
| download | postgresql-31510194cc9d87b355cb56e7d88c18c985d7a32a.tar.gz postgresql-31510194cc9d87b355cb56e7d88c18c985d7a32a.zip | |
Minor corrections for ALTER TYPE ADD VALUE IF NOT EXISTS patch.
Produce a NOTICE when the label already exists, for consistency with other
CREATE IF NOT EXISTS commands. Also, fix the code so it produces something
more user-friendly than an index violation when the label already exists.
This not incidentally enables making a regression test that the previous
patch didn't make for fear of exposing an unpredictable OID in the results.
Also some wordsmithing on the documentation.
Diffstat (limited to 'doc/src')
| -rw-r--r-- | doc/src/sgml/ref/alter_type.sgml | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/doc/src/sgml/ref/alter_type.sgml b/doc/src/sgml/ref/alter_type.sgml index 588887e1f9b..99c9d50429d 100644 --- a/doc/src/sgml/ref/alter_type.sgml +++ b/doc/src/sgml/ref/alter_type.sgml @@ -109,15 +109,16 @@ ALTER TYPE <replaceable class="PARAMETER">name</replaceable> ADD VALUE [ IF NOT <term><literal>ADD VALUE [ IF NOT EXISTS ] [ BEFORE | AFTER ]</literal></term> <listitem> <para> - This form adds a new value to an enum type. If the new value's place in - the enum's ordering is not specified using <literal>BEFORE</literal> or - <literal>AFTER</literal>, then the new item is placed at the end of the - list of values. + This form adds a new value to an enum type. The new value's place in + the enum's ordering can be specified as being <literal>BEFORE</literal> + or <literal>AFTER</literal> one of the existing values. Otherwise, + the new item is added at the end of the list of values. </para> <para> - If <literal>IF NOT EXISTS</literal> is used, it is not an error if the - type already contains the new value, and no action is taken. Otherwise, - an error will occur if the new value is already present. + If <literal>IF NOT EXISTS</literal> is specified, it is not an error if + the type already contains the new value: a notice is issued but no other + action is taken. Otherwise, an error will occur if the new value is + already present. </para> </listitem> </varlistentry> |