Re: Proposal : REINDEX xxx VERBOSE

From: Sawada Masahiko <sawada(dot)mshk(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Fabrízio Mello <fabriziomello(at)gmail(dot)com>, Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Proposal : REINDEX xxx VERBOSE
Date: 2015-05-05 15:10:33
Message-ID: CAD21AoBVVSO4mMc7s8j6_8gqVD6YyC8_u85wskRRNf_v-ubxvQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, May 1, 2015 at 9:04 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Thu, Apr 30, 2015 at 11:05 PM, Sawada Masahiko <sawada(dot)mshk(at)gmail(dot)com> wrote:
>> VACUUM has both syntax: with parentheses and without parentheses.
>> I think we should have both syntax for REINDEX like VACUUM does
>> because it would be pain to put parentheses whenever we want to do
>> REINDEX.
>> Are the parentheses optional in REINDEX command?
>
> No. The unparenthesized VACUUM syntax was added back before we
> realized that that kind of syntax is a terrible idea. It requires
> every option to be a keyword, and those keywords have to be in a fixed
> order. I believe the intention is to keep the old VACUUM syntax
> around for backward-compatibility, but not to extend it. Same for
> EXPLAIN and COPY.

REINDEX will have only one option VERBOSE for now.
Even we're in a situation like that it's not clear to be added newly
additional option to REINDEX now, we should need to put parenthesis?
Also I'm not sure that both implementation and documentation regarding
VERBOSE option should be optional.

Regards,

-------
Sawada Masahiko

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2015-05-05 15:40:30 Re: INSERT ... ON CONFLICT UPDATE/IGNORE 4.0
Previous Message Magnus Hagander 2015-05-05 13:57:28 Re: feature freeze and beta schedule