From: Fabien COELHO <coelho@cri.ensmp.fr>
To: Bruce Momjian <bruce@momjian.us>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Subject: Re: PG 13 release notes, first draft
Date: Thu, 7 May 2020 08:29:55 +0200 (CEST)
Message-ID: <alpine.DEB.2.22.394.2005070807160.211403@pseudo> (raw)
In-Reply-To: <20200506230750.GA3649@momjian.us>
References: <20200505031600.GB21752@momjian.us>
<alpine.DEB.2.22.394.2005050710540.175044@pseudo>
<20200505140055.GC21752@momjian.us>
<alpine.DEB.2.22.394.2005051811320.2183756@pseudo>
<20200506230750.GA3649@momjian.us>
Hello Bruce,
>>>> * "DOCUMENT THE DEFAULT GENERATION METHOD"
>>>> => The default is still to generate data client-side.
>>>
>>> My point is that the docs are not clear about this.
>>
>> Indeed.
>>
>>> Can you fix it?
>>
>> Sure. Attached patch adds an explicit sentence about it, as it was only
>> hinted about in the default initialization command string, and removes a
>> spurious empty paragraph found nearby.
>
> Thanks, patch applied.
Ok.
You might remove the "DOCUMENT THE DEFAULT…" in the release note.
I'm wondering about the commit comment: "Reported-by: Fabien COELHO",
actually you reported it, not me!
After looking again at the release notes, I do really think that
significant documentation changes do not belong to the "Source code"
section but should be in separate "Documentation" section, and that more
items should be listed there, because they represent a lot of not-so-fun
work, especially Tom's restructuration of tables, and possibly others.
About pgbench, ISTM that d37ddb745be07502814635585cbf935363c8a33d is worth
mentionning because it is a user-visible change.
--
Fabien.
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: coelho@cri.ensmp.fr, bruce@momjian.us
Subject: Re: PG 13 release notes, first draft
In-Reply-To: <alpine.DEB.2.22.394.2005070807160.211403@pseudo>
* 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