| 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 |
| 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 |