Re: s/LABEL/VALUE/ for ENUMs

On Tue, Nov 23, 2010 at 2:01 PM, David E. Wheeler <david@kineticode.com> wrote:
> On Nov 22, 2010, at 6:03 PM, Josh Berkus wrote:
>
>> ... "original patch".  Sorry.  Let's not fiddle with the names.
>
> To be clear, as things stand now, the new command is:
>
>    ALTER TYPE name ADD VALUE new_enum_value [ { BEFORE | AFTER } existing_enum_value ]
>
> So while the term in the SQL statement is "VALUE," it's called a "label" in the documentation. I think that's
confusing.Does anyone else? 

Yes.  As between the two options, I favor changing the command.  And
let's also paint it pink.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company