From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Sawada Masahiko <sawada(dot)mshk(at)gmail(dot)com> |
Cc: | 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-02-02 15:32:53 |
Message-ID: | 16229.1422891173@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Sawada Masahiko <sawada(dot)mshk(at)gmail(dot)com> writes:
> On Mon, Feb 2, 2015 at 9:21 PM, Michael Paquier
> <michael(dot)paquier(at)gmail(dot)com> wrote:
>> Now, I think that it may
>> be better to provide the keyword VERBOSE before the type of object
>> reindexed as REINDEX [ VERBOSE ] object.
> Actually, my first WIP version of patch added VERBOSE word at before
> type of object.
> I'm feeling difficult about that the position of VERBOSE word in
> REINDEX statement.
The way that FORCE was added to REINDEX was poorly thought out; let's not
double down on that with another option added without any consideration
for future expansion. I'd be happier if we adopted something similar to
the modern syntax for VACUUM and EXPLAIN, ie, comma-separated options in
parentheses.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2015-02-02 15:43:57 | Re: Release note bloat is getting out of hand |
Previous Message | Andres Freund | 2015-02-02 15:27:06 | POC: Cache data in GetSnapshotData() |