Re: Remove the refint contrib module (for v20)

From: Ayush Tiwari <ayushtiwari(dot)slg01(at)gmail(dot)com>
To: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
Cc: Álvaro Herrera <alvherre(at)kurilemu(dot)de>, Nathan Bossart <nathandbossart(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Daniel Gustafsson <daniel(at)yesql(dot)se>
Subject: Re: Remove the refint contrib module (for v20)
Date: 2026-06-29 08:02:02
Message-ID: CAJTYsWVMX_csjBafWngAqAAXBa=+Uy+rY2Je4jM6CJ11tF38Mw@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Mon, 29 Jun 2026 at 11:53, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:

> On Sat, Jun 6, 2026 at 4:28 AM Ayush Tiwari <ayushtiwari(dot)slg01(at)gmail(dot)com>
> wrote:
> > 0002 (test-only) teaches AdjustUpgrade.pm to drop the refint
> > extension and its leftover tables (pkeys, fkeys, fkeys2) from
> > contrib_regression_spi / regression_spi when upgrading from a version
> > that still shipped refint, so cross-version pg_upgrade tests don't
> > fail dump comparison once v20 opens.
> > I followed the existing < 19 btree_gist block style.
>
> In make-based contrib tests with USE_MODULE_DB=1, contrib/spi uses
> contrib_regression_autoinc database, since autoinc is the first entry
> in MODULES. So, in 0002 patch, shouldn't we also remove
> the refint-related objects from contrib_regression_autoinc?
>

Thanks for the review, Fujii-san!

Yes, you are right, I missed removing that in 0002 patch.

v5-0001 is identical to v4-0001.

Adding you suggested changes, in v5.

Regards,
Ayush

>

Attachment Content-Type Size
v5-0001-Remove-the-refint-contrib-module.patch application/octet-stream 39.9 KB
v5-0002-Handle-refint-removal-in-cross-version-pg_upgrade.patch application/octet-stream 2.1 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Ayush Tiwari 2026-06-29 08:05:25 Re: Remove the refint contrib module (for v20)
Previous Message JoongHyuk Shin 2026-06-29 07:59:40 Re: [PATCH] Don't call ereport(ERROR) from recovery target GUC assign hooks