agora inbox for pgsql-sql@postgresql.org  
help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Karsten Hilbert <Karsten.Hilbert@gmx.net>
Cc: pgsql-sql@postgresql.org, pgsql-general <pgsql-general@postgresql.org>
Subject: Re: [GENERAL] Q: documentation bug ?
Date: Thu, 04 Feb 2016 11:01:19 -0500
Message-ID: <17541.1454601679@sss.pgh.pa.us> (raw)
In-Reply-To: <20160204110532.GG22591@hermes.hilbert.loc>
References: <20160204105318.GD22591@hermes.hilbert.loc>
	<56B32F5D.5090009@2ndquadrant.fr>
	<20160204110532.GG22591@hermes.hilbert.loc>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-sql>

Karsten Hilbert <Karsten.Hilbert@gmx.net> writes:
> On Thu, Feb 04, 2016 at 12:00:45PM +0100, Vik Fearing wrote:
>>> REINDEX [ ( { VERBOSE } [, ...] ) ] { INDEX | TABLE | SCHEMA | DATABASE | SYSTEM } name

>>> Does this constitute a bug in the documentation or in the Debian version of PG ?

>> Neither.  It's a little bit obscure because {} and [] mean special
>> things, but () doesn't.

> Eagle-eyed !

Seems to me the syntax BNF here is bringing the problem on itself by using
{ ... } when in fact only one alternative is available.  I don't find
{ FOO | BAR } especially hard to read, but { FOO } is confusing because
you expect the {}'s to mean something and they really do not.

I'd be inclined to reduce this to

REINDEX [ ( VERBOSE ) ] { INDEX | TABLE | SCHEMA | DATABASE | SYSTEM } name

We can put back the extra decoration when and if a second option arrives.

			regards, tom lane


-- 
Sent via pgsql-sql mailing list (pgsql-sql@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-sql



view thread (6+ messages)  latest in thread

Message-ID: <17541.1454601679@sss.pgh.pa.us>
Permalink:  ../17541.1454601679@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/17541.1454601679@sss.pgh.pa.us

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-sql@postgresql.org
  Cc: tgl@sss.pgh.pa.us, Karsten.Hilbert@gmx.net, pgsql-general@postgresql.org
  Subject: Re: [GENERAL] Q: documentation bug ?
  In-Reply-To: <17541.1454601679@sss.pgh.pa.us>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox