Jim Nasby <jim.na...@bluetreble.com> writes:
> On 2/3/15 9:20 AM, Tom Lane wrote:
>> Well, the object type is not an optional part of the command.  It's
>> *necessary*.  I was thinking more like
>> REINDEX { INDEX | TABLE | etc } name [ ( option [, option ...] ) ]

> VACUUM puts the options before the table name, so ISTM it'd be best to 
> keep that with REINDEX. Either REINDEX (options) {INDEX | ...} or 
> REINDEX {INDEX | ...} (options).

Well, I really really don't like the first of those.  IMO the command name
is "REINDEX INDEX" etc, so sticking something in the middle of that is
bogus.

                        regards, tom lane


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to