Re: Proposal : REINDEX xxx VERBOSE - Mailing list pgsql-hackers

From Fabrízio de Royes Mello
Subject Re: Proposal : REINDEX xxx VERBOSE
Date
Msg-id CAFcNs+rsc+D=G5S3CWqUG7CEWMjpGdG2JKhOcrCspwPh1CLnAw@mail.gmail.com
Whole thread Raw
In response to Re: Proposal : REINDEX xxx VERBOSE  (Sawada Masahiko <sawada.mshk@gmail.com>)
List pgsql-hackers
<div dir="ltr"><div class="gmail_extra"><br />On Tue, Feb 3, 2015 at 9:09 AM, Sawada Masahiko <<a
href="mailto:sawada.mshk@gmail.com">sawada.mshk@gmail.com</a>>wrote:<br />><br />> On Tue, Feb 3, 2015 at
12:32AM, Tom Lane <<a href="mailto:tgl@sss.pgh.pa.us">tgl@sss.pgh.pa.us</a>> wrote:<br />> > Sawada
Masahiko<<a href="mailto:sawada.mshk@gmail.com">sawada.mshk@gmail.com</a>> writes:<br />> >> On Mon, Feb
2,2015 at 9:21 PM, Michael Paquier<br />> >> <<a
href="mailto:michael.paquier@gmail.com">michael.paquier@gmail.com</a>>wrote:<br />> >>> Now, I think
thatit may<br />> >>> be better to provide the keyword VERBOSE before the type of object<br />>
>>>reindexed as REINDEX [ VERBOSE ] object.<br />> ><br />> >> Actually, my first WIP version
ofpatch added VERBOSE word at before<br />> >> type of object.<br />> >> I'm feeling difficult about
thatthe position of VERBOSE word in<br />> >> REINDEX statement.<br />> ><br />> > The way that
FORCEwas added to REINDEX was poorly thought out; let's not<br />> > double down on that with another option
addedwithout any consideration<br />> > for future expansion.  I'd be happier if we adopted something similar
to<br/>> > the modern syntax for VACUUM and EXPLAIN, ie, comma-separated options in<br />> >
parentheses.<br/>> ><br />><br />> I understood.<br />> I'm imagining new REINDEX syntax are
followings.<br/>> - REINDEX (INDEX, VERBOSE) hoge_idx;<br />> - REINDEX (TABLE) hoge_table;<br />><br />>
i.g.,I will add following syntax format,<br />> REINDEX ( { INDEX | TABLE | SCHEMA | SYSTEM | DATABASE } , [VERBOSE]
)<br/>> name [FORCE];<br />><br />> Thought?<br />><br /><br /></div><div class="gmail_extra">I don't think
thekeyworks INDEX, TABLE, SCHEMA, SYSTEM and DATABASE are options... they are part of the command IMHO.<br /><br
/></div><divclass="gmail_extra">Maybe something like:<br /><br /><br />REINDEX { INDEX | TABLE | SCHEMA | SYSTEM |
DATABASE} [( [ FORCE ], [VERBOSE] ) ] name;<br /><br /></div><div class="gmail_extra">And maintain the old syntax for
compatibilityof course.<br /></div><div class="gmail_extra"><br /></div><div class="gmail_extra">Regards,<br /><br
/>--<br/>Fabrízio de Royes Mello<br />Consultoria/Coaching PostgreSQL<br />>> Timbira: <a
href="http://www.timbira.com.br">http://www.timbira.com.br</a><br/>>> Blog: <a
href="http://fabriziomello.github.io">http://fabriziomello.github.io</a><br/>>> Linkedin: <a
href="http://br.linkedin.com/in/fabriziomello">http://br.linkedin.com/in/fabriziomello</a><br/>>> Twitter: <a
href="http://twitter.com/fabriziomello">http://twitter.com/fabriziomello</a><br/>>> Github: <a
href="http://github.com/fabriziomello">http://github.com/fabriziomello</a></div></div>

pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: Release note bloat is getting out of hand
Next
From: Robert Haas
Date:
Subject: Re: Release note bloat is getting out of hand