pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Justin Pryzby <pryzby@telsasoft.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Jonathan S. Katz <jkatz@postgresql.org>
Cc: Alvaro Herrera <alvherre@alvh.no-ip.org>
Cc: Nathan Bossart <nathandbossart@gmail.com>
Cc: Bruce Momjian <bruce@momjian.us>
Cc: pgsql-hackers@postgresql.org, David Rowley <dgrowleyml@gmail.com>
Subject: Re: First draft of the PG 15 release notes
Date: Sun, 25 Sep 2022 16:50:09 -0500
Message-ID: <20220925215009.GC21938@telsasoft.com> (raw)
In-Reply-To: <2134470.1663954387@sss.pgh.pa.us>
References: <20220913111333.c56vyzgipzzyzimu@alvherre.pgsql>
	<a6661e2c-72e0-b4bd-9301-9225bdddda4c@postgresql.org>
	<2121704.1663946744@sss.pgh.pa.us>
	<24c8bb66-73ab-199b-b99a-21d8d7e0b273@postgresql.org>
	<2134470.1663954387@sss.pgh.pa.us>

On Fri, Sep 23, 2022 at 01:33:07PM -0400, Tom Lane wrote:
> "Jonathan S. Katz" <jkatz@postgresql.org> writes:
> > On 9/23/22 11:25 AM, Tom Lane wrote:
> >> I'm planning to do a final(?) pass over the v15 notes today,
> >> but I thought it'd be appropriate to push this separately.
> 
> > RE "final pass", there's still an errant "BACKPATCHED"[1] that still 
> > needs addressing. I didn't have a chance to verify if it was indeed 
> > backpatched.
> 
> Yeah, that one indeed needs removed (and I've done so).  I see a
> few other places where Bruce left notes about things that need more
> clarification.  I'm just finishing a pass of "update for subsequent
> commits", and then I'll start on copy-editing.

Some possible changes for your consideration.

+        Store <application>pg_upgrade</application>'s log and
+        temporary files in a subdirectory of the new cluster called
+        <filename>pg_upgrade_output.d</filename> (Justin Pryzby)

+        Previously such files were left in the current directory,
+        requiring manual cleanup.  It's still necessary to remove them
+        manually afterwards, but now one can just remove that whole
+        subdirectory.

If pg_upgrade succeeds, then it removes the dir itself (so it's not
"necessary").

And if it fails after starting to restore the schema, then it's
necessary to remove not the "subdirectory" but the whole new-cluster
dir.

+        Make <application>pg_upgrade</application> preserve tablespace
+        and database OIDs, as well as table relfilenode numbers

s/table/relation/ ?

You changed this to use spaces:
|        The new setting is <literal>log_destination = jsonlog</literal>.
but then left these without spaces:
|        and <literal>wal_level=minimal</literal>.
|        This is enabled via <literal>--compress=lz4</literal> and requires

+       value, use the transaction start time not wall clock time to

s/not/rather than/ ?

+        Adjust <application>psql</application> so that Readline's

should use <productname>Readline ?

+        Previously a pound marker was inserted, but that's pretty
+        unhelpful in SQL.

This sounds more like a candid commit message than a release note.

+        Improve performance of dumping databases with many objects

s/of/when/ ?

+       New options are <literal>server</literal> to write the

*The* new options

+        In some cases a partition child table could appear more than once.

Technically "partition child table" is redundant

-- 
Justin





view thread (145+ messages)  latest in thread

Message-ID: <20220925215009.GC21938@telsasoft.com>
Permalink:  ../20220925215009.GC21938@telsasoft.com/
Also on:    postgresql.org/message-id/20220925215009.GC21938@telsasoft.com

 · 

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-hackers@postgresql.org
  Cc: pryzby@telsasoft.com, tgl@sss.pgh.pa.us, jkatz@postgresql.org, alvherre@alvh.no-ip.org, nathandbossart@gmail.com, bruce@momjian.us, dgrowleyml@gmail.com
  Subject: Re: First draft of the PG 15 release notes
  In-Reply-To: <20220925215009.GC21938@telsasoft.com>

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

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