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)



Reply via email to