Re: First draft of the PG 15 release notes

Поиск
Список
Период
Сортировка
От Justin Pryzby
Тема Re: First draft of the PG 15 release notes
Дата
Msg-id 20220925215009.GC21938@telsasoft.com
обсуждение исходный текст
Ответ на Re: First draft of the PG 15 release notes  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: First draft of the PG 15 release notes
Список pgsql-hackers
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



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Hamid Akhtar
Дата:
Сообщение: Re: add checkpoint stats of snapshot and mapping files of pg_logical dir
Следующее
От: Justin Pryzby
Дата:
Сообщение: Re: add checkpoint stats of snapshot and mapping files of pg_logical dir