From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Explicitly list dependent types as extension members in pg_depen |
Date: | 2024-03-04 19:49:48 |
Message-ID: | E1rhEJn-0024wH-TS@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Explicitly list dependent types as extension members in pg_depend.
Auto-generated array types, multirange types, and relation rowtypes
are treated as dependent objects: they can't be dropped separately
from the base object, nor can they have their own ownership or
permissions. We previously felt that, for objects that are in an
extension, only the base object needs to be listed as an extension
member in pg_depend. While that's sufficient to prevent inappropriate
drops, it results in undesirable answers if someone asks whether a
dependent type belongs to the extension. It looks like the dependent
type is just some random separately-created object that happens to
depend on the base object. Notably, this results in postgres_fdw
concluding that expressions involving an array type are not shippable
to the remote server, even when the defining extension has been
whitelisted.
To fix, cause GenerateTypeDependencies to make extension dependencies
for dependent types as well as their base objects, and adjust
ExecAlterExtensionContentsStmt so that object addition and removal
operations recurse to dependent types. The latter change means that
pg_upgrade of a type-defining extension will end with the dependent
type(s) now also listed as extension members, even if they were
not that way in the source database. Normally we want pg_upgrade
to precisely reproduce the source extension's state, but it seems
desirable to make an exception here.
This is arguably a bug fix, but we can't back-patch it since it
causes changes in the expected contents of pg_depend. (Because
it does, I've bumped catversion, even though there's no change
in the immediate post-initdb catalog contents.)
Tom Lane and David Geier
Discussion: https://postgr.es/m/4a847c55-489f-4e8d-a664-fc6b1cbe306f@gmail.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/e5bc9454e527b1cba97553531d8d4992892fdeef
Modified Files
--------------
src/backend/catalog/pg_type.c | 17 ++-
src/backend/commands/extension.c | 83 +++++++++--
src/include/catalog/catversion.h | 2 +-
src/test/modules/test_extensions/Makefile | 3 +-
.../test_extensions/expected/test_extensions.out | 159 +++++++++++++++++++--
src/test/modules/test_extensions/meson.build | 2 +
.../test_extensions/sql/test_extensions.sql | 13 ++
.../modules/test_extensions/test_ext9--1.0.sql | 8 ++
src/test/modules/test_extensions/test_ext9.control | 3 +
9 files changed, 265 insertions(+), 25 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Nathan Bossart | 2024-03-04 21:43:58 | pgsql: Add macro for customizing an archiving WARNING message. |
Previous Message | Robert Haas | 2024-03-04 19:46:58 | pgsql: pg_verifybackup: Refactor parse_manifest_file. |