From: Jonathan S. Katz <jkatz@postgresql.org>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Masahiko Sawada <masahiko.sawada@2ndquadrant.com>
Cc: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Cc: Bruce Momjian <bruce@momjian.us>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Subject: Re: PG 13 release notes, first draft
Date: Tue, 15 Sep 2020 11:56:40 -0400
Message-ID: <ad54fe2c-8692-d538-3406-b246343c2198@postgresql.org> (raw)
In-Reply-To: <1305217.1600184757@sss.pgh.pa.us>
References: <20200505031600.GB21752@momjian.us>
<448a382b-ae07-3126-5a08-aacda9aa28ea@postgresql.org>
<f1c43223-3987-a23f-2063-18fd0aa4f0d4@2ndquadrant.com>
<CA+fd4k606sc9y-hcmyqSRdEZMoa6uMAHSLFCdFa4bC_W4FkbwQ@mail.gmail.com>
<63d3f3aa-e42d-dd3d-643c-2e91a13a0762@postgresql.org>
<e0a4d177-d003-8ebb-5296-5a445472b66f@postgresql.org>
<1305217.1600184757@sss.pgh.pa.us>
On 9/15/20 11:45 AM, Tom Lane wrote:
> "Jonathan S. Katz" <jkatz@postgresql.org> writes:
>> On a different note, I became aware of this[1] and noticed that dropping
>> "CREATE EXTENSION ... FROM" was not listed in the incompatibilities
>> section, so proposing the attached. I have no strong opinions on the
>> final wording, mainly wanted to get it listed.
>
> It is listed already in the "Additional Modules" section (about line 2940
> in release-13.sgml as of right now).
...sort of. It talks about the feature, but does not talk about the
syntax removal, which is what I was originally searching for in the
release notes.
> I don't know Bruce's exact reasoning
> for not including it in the "Incompatibilities" section, but I tend to
> agree that it shouldn't be significant to any real-world user.
I do tend agree with this intuitively but don't have any data to back it
up either way. That said, we did modify the command and it would be good
to at least mention the fact we dropped "FROM" somewhere in the release
notes. It provides a good reference in case someone reports an "issue"
in the future stemming from dropping the "FROM" keyword, we can say "oh
it changed in PG13, see XYZ."
(Speaking from having used the release notes to perform similar
troubleshooting).
> I think
> that the postgis testing issue you reference is just an ancient test
> case that they should drop as no longer relevant.
Sure, and AIUI they're going to do that, mostly that was a reference
point to the changed syntax.
Jonathan
Attachments:
signature.asc (application/pgp-signature, 832B) download
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: jkatz@postgresql.org, tgl@sss.pgh.pa.us, masahiko.sawada@2ndquadrant.com, peter.eisentraut@2ndquadrant.com, bruce@momjian.us
Subject: Re: PG 13 release notes, first draft
In-Reply-To: <ad54fe2c-8692-d538-3406-b246343c2198@postgresql.org>
* 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