Re: Proposal : REINDEX xxx VERBOSE

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>
Cc: sawada(dot)mshk(at)gmail(dot)com, Jim(dot)Nasby(at)bluetreble(dot)com, michael(dot)paquier(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Proposal : REINDEX xxx VERBOSE
Date: 2015-02-05 01:24:49
Message-ID: 18004.1423099489@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> writes:
> The phrase "{INDEX | TABLE |..} name" seems to me indivisible as
> target specification. IMHO, the options for VACUUM and so is
> placed *just after* command name, not *before* the target.

> If this is right, the syntax would be like this.

> REINDEX [ (option [, option ...] ) ] {INDEX | TABLE | etc } name

> What do you think about this?

I think this is wrong and ugly. INDEX etc are part of the command name.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2015-02-05 01:33:12 Re: binworld and install-binworld targets - was Re: Release note bloat is getting out of hand
Previous Message Andrew Dunstan 2015-02-05 01:20:49 Re: binworld and install-binworld targets - was Re: Release note bloat is getting out of hand