Re: Postgresql 18beta1 and SPI changes

From: Achilleas Mantzios <a(dot)mantzios(at)cloud(dot)gatewaynet(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "pgsql-general(at)lists(dot)postgresql(dot)org" <pgsql-general(at)lists(dot)postgresql(dot)org>, itdev(at)gatewaynet(dot)com
Subject: Re: Postgresql 18beta1 and SPI changes
Date: 2025-05-12 12:50:12
Message-ID: 59659764-5790-430c-b38d-e4ecdd4a744a@cloud.gatewaynet.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general

Dear All, Dear Tom

On 5/11/25 16:20, Tom Lane wrote:

> Achilleas Mantzios<a(dot)mantzios(at)cloud(dot)gatewaynet(dot)com> writes:
>> We use are own version of DBmirror, we run our replication in a highly
>> fine grained manner. So every upgrade I have to make the code compile
>> and test. Up to PostgreSQL 17, I only got minor compilation problems
>> that I managed to resolve fairly easily. However this didn't prove to be
>> the case with PostgreSQL 18beta1, it proved harder to compile and as my
>> fears were verified, it has serious problems.
>> My question : is 18's SPI stabilized ? Can I start work on our version
>> of DBmirror ? Or wait for 18beta2 or -RC ?
> If you think there are changes we need to make, you'd better get
> specific sooner not later. I'm not aware of any large fixes that
> are pending, cf
>
> https://wiki.postgresql.org/wiki/PostgreSQL_18_Open_Items

I attach

a) our old source (pending.c.orig), as of PostgreSQL 17 (tested for some
7 months, so pretty well tested),

b) the compilation errors when compiled against 18beta1, and

c) the patch that I came up with, which seems (in my minimal testing) to
yield correct results on 18beta1.

The majority of serious warnings have to do with de-toasting arrays and
the PK's int2vector , while the error has to do with getting column
details such as attisdropped and attname.

Please have a look, and share your thoughts. I haven't touched serious C
coding till I first wrote the above sometime in 2004 with a bunch of
additions some years ago.

>
> regards, tom lane

Attachment Content-Type Size
pending.c.orig text/plain 60.8 KB
make.log text/x-log 9.0 KB
pending_changes_against_18beta1.patch text/x-patch 3.8 KB

In response to

Responses

Browse pgsql-general by date

  From Date Subject
Next Message Markus Demleitner 2025-05-12 14:12:17 Re: Index not used in certain nested views but not in others
Previous Message Tatsuo Ishii 2025-05-12 00:21:31 Re: Proposed Japanese Translation of Code of Conduct Policy