From: Stephen Frost <sfrost@snowman.net>
To: Magnus Hagander <magnus@hagander.net>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Cc: Justin Pryzby <pryzby@telsasoft.com>
Cc: Heikki Linnakangas <hlinnaka@iki.fi>
Cc: Ian Lawrence Barwick <barwick@gmail.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Subject: Re: [PATCH] remove deprecated v8.2 containment operators
Date: Fri, 13 Nov 2020 10:03:43 -0500
Message-ID: <20201113150343.GT16415@tamriel.snowman.net> (raw)
In-Reply-To: <CABUevEyLtOAPtY3MYap0b6H23WmsrGHmFN6On6v7CDi14FORDw@mail.gmail.com>
References: <20201027032511.GF9241@telsasoft.com>
<155ecfd8-7ac6-1875-5090-d79a586a8847@enterprisedb.com>
<235561.1605220100@sss.pgh.pa.us>
<CABUevEyLtOAPtY3MYap0b6H23WmsrGHmFN6On6v7CDi14FORDw@mail.gmail.com>
Greetings,
* Magnus Hagander (magnus@hagander.net) wrote:
> On Thu, Nov 12, 2020 at 11:28 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> > > The changes to the contrib modules appear to be incomplete in some ways.
> > > In cube, hstore, and seg, there are no changes to the extension
> > > scripts to remove the operators. All you're doing is changing the C
> > > code to no longer recognize the strategy, but that doesn't explain what
> > > will happen if the operator is still used. In intarray, by contrast,
> > > you're editing an existing extension script, but that should be done by
> > > an upgrade script instead.
> >
> > In the contrib modules, I'm afraid what you gotta do is remove the
> > SQL operator definitions but leave the opclass code support in place.
> > That's because there's no guarantee that users will update the extension's
> > SQL version immediately, so a v14 build of the .so might still be used
> > with the old SQL definitions. It's not clear how much window we need
> > give for people to do that update, but I don't think "zero" is an
> > acceptable answer.
>
> Based on my experience from the field, the answer is "never".
>
> As in, most people have no idea they are even *supposed* to do such an
> upgrade, so they don't do it. Until we solve that problem, I think
> we're basically stuck with keeping them "forever". (and even if/when
> we do, "zero" is probably not going to cut it, no)
Yeah, this is a serious problem and one that we should figure out a way
to fix or at least improve on- maybe by having pg_upgrade say something
about extensions that could/should be upgraded..?
Thanks,
Stephen
Attachments:
signature.asc (application/pgp-signature, 818B) 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: sfrost@snowman.net, magnus@hagander.net, tgl@sss.pgh.pa.us, peter.eisentraut@enterprisedb.com, pryzby@telsasoft.com, hlinnaka@iki.fi, barwick@gmail.com
Subject: Re: [PATCH] remove deprecated v8.2 containment operators
In-Reply-To: <20201113150343.GT16415@tamriel.snowman.net>
* 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