Hi,
I realized that ALTER EXTENSION foo ADD TYPE _bar does pretty much the
same via ExecAlterExtensionContentsStmt(). So the code in the patch
seems fine.
On 1/8/24 12:21, David Geier wrote:
The attached patch just adds a 2nd dependency between the array type
and the extension, using recordDependencyOnCurrentExtension(). It
seems like that the other internal dependency on the element type must
stay? If that seems reasonable I can add a test to
modules/test_extensions. Can extensions in contrib use test extension
in their own tests? It looks like postgres_fdw doesn't test any of the
shippability logic.
--
David Geier
(ServiceNow)