Re: CLUSTER ALL syntax

Alvaro Herrera <alvherre@dcc.uchile.cl> writes:
> What I don't understand is what are the parameters in the
> ReindexDatabase function for.  For example, the boolean all is always
> false in tcop/utility.c (and there are no other places that the function
> is called).  Also, the database name is checked to be equal to a
> "constant" value, the database name that the standalone backend is
> connected to.  Why are those useful?

Well, passing all=true would implement REINDEX ALL ...

As for the database name, we could perhaps change the syntax to just
REINDEX DATABASE; not sure if it's worth the trouble.
        regards, tom lane