PostGIS and json_categorize_type (Re: pgsql: Revert SQL/JSON features)

From: Christoph Berg <myon(at)debian(dot)org>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org, PostGIS Development Discussion <postgis-devel(at)lists(dot)osgeo(dot)org>
Subject: PostGIS and json_categorize_type (Re: pgsql: Revert SQL/JSON features)
Date: 2022-09-07 09:01:22
Message-ID: Yxhd4gw1xJP+SsDe@msg.df7cb.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Re: Andrew Dunstan
> Revert SQL/JSON features
>
> The reverts the following and makes some associated cleanups:

-void
+static void
json_categorize_type(Oid typoid,
JsonTypeCategory *tcategory,
Oid *outfuncoid)

This chunk broke PostGIS 3.3.0 compiled with 15beta3, when used with
15beta4:

psql -Xc 'CREATE EXTENSION postgis'
ERROR: could not load library "/usr/lib/postgresql/15/lib/postgis-3.so": /usr/lib/postgresql/15/lib/postgis-3.so: undefined symbol: json_categorize_type

The PostGIS source has this comment:

* The following code was all cut and pasted directly from
* json.c from the Postgres source tree as of 2019-03-28.
* It would be far better if these were exported from the
* backend so we could just use them here. Maybe someday.
* Sequel: 2022-04-04 That some day finally came in PG15
...
#if POSTGIS_PGSQL_VERSION < 170
static void
json_categorize_type(Oid typoid,
JsonTypeCategory *tcategory,
Oid *outfuncoid)

The "< 17" part was added on 2022-09-03, probably because of this
breakage.

Recompiling the (unmodified) 3.3.0 against 15beta4 seems to fix the
problem.

So, there is probably no issue here, but I suggest this "static" might
be considered to be removed again so PostGIS can use it.

Christoph

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Christoph Berg 2022-09-07 09:07:35 Re: PostGIS and json_categorize_type (Re: pgsql: Revert SQL/JSON features)
Previous Message David Rowley 2022-09-07 03:47:21 pgsql: Make more effort to put a sentinel at the end of allocated memor

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2022-09-07 09:04:02 Re: Handle infinite recursion in logical replication setup
Previous Message Kyotaro Horiguchi 2022-09-07 08:58:22 Re: Tracking last scan time