Date: Wed, 22 Nov 2023 16:40:44 -0500
From: Bruce Momjian <bruce@momjian.us>
To: ap@robillo.net
Cc: pgsql-docs@postgresql.org
Subject: Re: [DOCS] Double negation in parameter description
Message-ID: <ZV51XMu5dE20Pmrr@momjian.us>
References: <20170208152743.1411.6073@wrigleys.postgresql.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="XylPLp8AahzGwdb6"
Content-Disposition: inline
In-Reply-To: <20170208152743.1411.6073@wrigleys.postgresql.org>
Archived-At: 
 <https://www.postgresql.org/message-id/ZV51XMu5dE20Pmrr%40momjian.us>
Precedence: bulk


--XylPLp8AahzGwdb6
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Wed, Feb  8, 2017 at 03:27:43PM +0000, ap@robillo.net wrote:
> The following documentation comment has been logged on the website:
> 
> Page: https://www.postgresql.org/docs/9.6/static/sql-refreshmaterializedview.html
> Description:
> 
> In the description of the CONCURRENTLY parameter (3rd line/paragraph, the
> statement employs double negation which makes it hard to read and
> understand.
> 
> &quot;This option may not be used when the materialized view is not already
> populated.&quot;
> 
> I guess the author would want to mean: 
> 
> &quot;This option may only be used on a populated materialized view.&quot;?
> 
> Your clarification is much appreciated.

I know this is six years old, but fixed in master with the attached
patch.

-- 
  Bruce Momjian  <bruce@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Only you can decide what is important to you.

--XylPLp8AahzGwdb6
Content-Type: text/x-diff; charset=us-ascii
Content-Disposition: attachment; filename="master.diff"

diff --git a/doc/src/sgml/ref/refresh_materialized_view.sgml b/doc/src/sgml/ref/refresh_materialized_view.sgml
index 675d6090f3..7a019162c3 100644
--- a/doc/src/sgml/ref/refresh_materialized_view.sgml
+++ b/doc/src/sgml/ref/refresh_materialized_view.sgml
@@ -67,7 +67,7 @@ REFRESH MATERIALIZED VIEW [ CONCURRENTLY ] <replaceable class="parameter">name</
       expression index or include a <literal>WHERE</literal> clause.
      </para>
      <para>
-      This option may not be used when the materialized view is not already
+      This option can only be used when the materialized view is already
       populated.
      </para>
      <para>

--XylPLp8AahzGwdb6--