Re: MERGE PARTITIONS and DEPENDS ON EXTENSION.

From: "Matheus Alcantara" <matheusssilv97(at)gmail(dot)com>
To: "Dmitry Koval" <d(dot)koval(at)postgrespro(dot)ru>, "pgsql-hackers" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: MERGE PARTITIONS and DEPENDS ON EXTENSION.
Date: 2026-04-20 15:23:15
Message-ID: DHY3375Y4A28.33LLEFIAUQKI@gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun Apr 19, 2026 at 9:04 PM -03, Dmitry Koval wrote:
> Hi Matheus!
>
> >v3-0001-Preserve-extension-dependencies-on-indexes-during.patch
>
> I looked patch and did not find problems. But there is one point:
> extensions btree_gist, btree_gin, citext are not included by default.
> So command
>
> > ./configure --enable-debug --enable-cassert --prefix `pwd`/install
> > >/dev/null && make -s && make install -s && make check
>
> generates errors like
>
> ERROR: extension "btree_gist" is not available
> ERROR: extension "btree_gin" is not available
> ERROR: extension "citext" is not available
>
> Might be it would be better to use for tests extensions from the
> catalog src/test/modules/test_extensions (see test
> src/test/modules/test_extensions/sql/test_extdepend.sql)?
>

Thanks for looking at this!

Yeah, some build farm animals will not be happy with these new tests.
Fixed on new attached v4 to use extensions from
src/test/modules/test_extensions.

> >extension dependencies on parent partitioned indexes don't seem to
> >prevent DROP EXTENSION, but dependencies on child partition indexes
> >do. ...
>
> I agree, it looks strange ...
>

I'll start a new thread to discuss this.

--
Matheus Alcantara
EDB: https://www.enterprisedb.com

Attachment Content-Type Size
v4-0001-Preserve-extension-dependencies-on-indexes-during.patch text/plain 28.9 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Melanie Plageman 2026-04-20 15:48:09 Re: eliminate xl_heap_visible to reduce WAL (and eventually set VM on-access)
Previous Message Alvaro Herrera 2026-04-20 15:11:56 Re: Adding REPACK [concurrently]