Date: Wed, 19 May 2021 09:39:08 -0500
From: Justin Pryzby <pryzby@telsasoft.com>
To: Bruce Momjian <bruce@momjian.us>
Cc: pgsql-hackers@postgresql.org
Subject: Re: PG 14 release notes, first draft
Message-ID: <20210519143908.GU373@telsasoft.com>
References: <20210510060308.GC6088@momjian.us>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210510060308.GC6088@momjian.us>
User-Agent: Mutt/1.9.4 (2018-02-28)
Archived-At: 
 <https://www.postgresql.org/message-id/20210519143908.GU373%40telsasoft.com>
Precedence: bulk

These sound weird since markup was added in 6a5bde7d4:
https://www.postgresql.org/docs/devel/release-14.html
| Remove server and Chapter 34 support for the version 2 wire protocol (Heikki Linnakangas)
...
| Pass doubled quote marks in Chapter 36 SQL command strings literally (Tom Lane)

-Remove server and libpq support for the version 2 wire protocol (Heikki Linnakangas)
+Remove server and <xref linkend="libpq"/> support for the version 2 <link linkend="protocol">wire protocol</link> (Heikki Linnakangas)

> Force custom server variable names to match the pattern used for unquoted SQL identifiers (Tom Lane)
Say "Require" not force?

> This setting was disabled in PostgreSQL version 13.3.
"disabled" sounds like it was set to "off".  Maybe say it was ignored.

> Add long-running queries to be canceled if the client disconnects (Sergey Cherkashin, Thomas Munro)
Should say: Allow

> The server variable client_connection_check_interval allows supporting operating systems, e.g., Linux, to automatically cancel queries by disconnected clients.
say "some operating systems" ?

> This can be disabled by turning client option "sslsni" off.
"turning off"

| Add %P to log_line_prefix to report the parallel group leader (Justin Pryzby)

Maybe it should say "Allow %P in log_line_prefix to ...", otherwise it sounds
like the default was changed.

| Reduce the default value of vacuum_cost_page_miss (Peter Geoghegan) 
|  This new default better reflects current hardware capabilities. 
Also say: the previous default was 10.