Date: Wed, 11 Sep 2019 18:15:22 -0300
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Bruce Momjian <bruce@momjian.us>
Cc: Tomas Vondra <tomas.vondra@2ndquadrant.com>,
	PostgreSQL-documentation <pgsql-docs@postgresql.org>
Subject: Re: Most-common value docs in PG 12
Message-ID: <20190911211522.GA7762@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20190830232804.GA10645@momjian.us>
User-Agent: Mutt/1.9.4 (2018-02-28)
Precedence: bulk

On 2019-Aug-30, Bruce Momjian wrote:

> OK, how is this patch?  I didn't mention psql since I think everyone
> expects psql to show all information about tables and indexes.

Why would you change perform.sgml?  It seems unnecessary; the commands
shown work fine.

This part seems okay:

> diff --git a/doc/src/sgml/ref/create_statistics.sgml b/doc/src/sgml/ref/create_statistics.sgml
> index ec23a154d6..5b583aacb4 100644
> --- a/doc/src/sgml/ref/create_statistics.sgml
> +++ b/doc/src/sgml/ref/create_statistics.sgml
> @@ -98,7 +98,8 @@ CREATE STATISTICS [ IF NOT EXISTS ] <replaceable class="parameter">statistics_na
>      <listitem>
>       <para>
>        The name of a table column to be covered by the computed statistics.
> -      At least two column names must be given.
> +      At least two column names must be given;  the order of the column names
> +      is insignificant.
>       </para>
>      </listitem>
>     </varlistentry>

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services