MIME-Version: 1.0
Date: Sat, 20 May 2023 01:02:58 +1000
From: Justin Clift <justin@postgresql.org>
To: "Jonathan S. Katz" <jkatz@postgresql.org>
Cc: PostgreSQL Advocacy <pgsql-advocacy@lists.postgresql.org>
Subject: Re: PostgreSQL 16 Beta 1 release announcement draft
In-Reply-To: <b83f168a-8cc4-a121-e94d-5f30bb6ea6fa@postgresql.org>
References: <b83f168a-8cc4-a121-e94d-5f30bb6ea6fa@postgresql.org>
User-Agent: Roundcube Webmail/1.4.13
Message-ID: <47d8c7833c7bb98850448ca651cdf44d@postgresql.org>
Content-Type: text/plain; charset=US-ASCII;
 format=flowed
Content-Transfer-Encoding: 7bit
Archived-At: 
 <https://www.postgresql.org/message-id/47d8c7833c7bb98850448ca651cdf44d%40postgresql.org>
Precedence: bulk

On 2023-05-19 14:18, Jonathan S. Katz wrote:
> Hi,
> 
> Attached is a draft of the release announcement for PostgreSQL 16 Beta
> 1. The goal of this announcement is to get people excited about
> testing the beta and highlight many of the new features.
> 
> Please review for inaccuracies, omissions, and other suggestions / 
> errors.

Typo:

   "PostgreSQL 16 continues to give users to the ability grant privileged 
..."

Should be:

   "PostgreSQL 16 continues to give users the ability to grant privileged 
..."

Hmmm, upon re-reading that whole sentence (not just my fragment of it 
above)
seems pretty unwieldy.  A better one isn't springing to mind though. ;)

---

Should this:

   "Additionally, starting with release, logical replication ..."

Have the word "this" before "release"?

---

Typo:

   "... use the authenticated credentials to connected to other 
services."

Should be:

   "... use the authenticated credentials to connect to other services."

The sentence after that starts:

   "This release also adds ..."

With the one following it starting with "This includes ...".  Together,
they a bit unwieldy too.  Maybe something like this instead?

   "This release also adds several new security-oriented connection
    parameters for clients, including ..."

---

Would this:

   "You can now also set `sslrootcert` to `system` to specify that
    PostgreSQL can use the trusted certificate authority (CA) store
    for that operating system."

Be better as?

   "You can now also set `sslrootcert` to `system` to instruct
    PostgreSQL to use the trusted certificate authority (CA) store
    provided by the operating system."

---

Nothing else is jumping out at me.  It all seems pretty good. :)

+ Justin