Re: Re: pgsql: Fix assorted inconsistencies in GIN opclass support function dec

From: Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-committers <pgsql-committers(at)postgresql(dot)org>
Subject: Re: Re: pgsql: Fix assorted inconsistencies in GIN opclass support function dec
Date: 2016-05-03 18:47:41
Message-ID: CAPpHfdt=ASL5JbBqL9Va2Q4kN_dFzH0bpPm6jpch_VPycE3bOA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Tue, May 3, 2016 at 4:32 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru> writes:
> > On Tue, May 3, 2016 at 12:09 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> >> It doesn't really matter, though, because there simply isn't any need to
> >> refer to these functions from SQL.
>
> > What if we'll want to reuse some on these functions in new opclass?
>
> What context do you imagine that happening in? Surely no other extension
> would attempt to re-use them. If, say, hstore itself wanted to reuse
> these functions, that would be in a new version of hstore--n.n.sql, so
> there's no problem.
>

Let's consider particular example: gin_extract_hstore() function.

In PostgreSQL 9.6 hstore 1.3 it's gin_extract_hstore(hstore, internal)
RETURNS internal.
In PostgreSQL 9.5 hstore 1.3 it's gin_extract_hstore(internal, internal)
RETURNS internal.
After pg_upgrade 9.5 => 9.6 it will be still gin_extract_hstore(internal,
internal) RETURNS internal.

Thus, in 9.6 hstore 1.3 we can meet either

1. gin_extract_hstore(internal, internal) RETURNS internal
2. gin_extract_hstore(hstore, internal) RETURNS internal

Then, if we want to reuse this function in hstore 1.4, we will need to
write a migration script hstore--1.3--1.4.sql. How can we refer
gin_extract_hstore() function from this script?

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2016-05-04 01:10:14 pgsql: doc: Fix typos
Previous Message Robert Haas 2016-05-03 18:45:42 pgsql: Fix more things to be parallel-safe.

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2016-05-03 18:49:03 Re: ALTER TABLE lock downgrades have broken pg_upgrade
Previous Message Robert Haas 2016-05-03 18:45:58 Re: Accidentally parallel unsafe functions