From: Tom Lane <tgl@sss.pgh.pa.us>
To: "Regina Obe" <lr@pcorp.us>
cc: "'Sandro Santilli'" <strk@kbt.io>,
        "'Gregory Stark \(as CFM\)'" <stark.cfm@gmail.com>,
        pgsql-hackers@lists.postgresql.org
Subject: Re: Ability to reference other extensions by schema in extension scripts
In-reply-to: <007601d95c01$95744cd0$c05ce670$@pcorp.us>
References: <CAM-w4HP91M-r=h4nExwonHf03SRwKyNYnvFHX2FFoFMv0pRQBA@mail.gmail.com> <005401d95083$26ca24a0$745e6de0$@pcorp.us> <995823.1678478825@sss.pgh.pa.us> <004b01d95394$16123ee0$4236bca0$@pcorp.us> <1012975.1678486476@sss.pgh.pa.us> <005501d953a0$906c4df0$b144e9d0$@pcorp.us> <1019515.1678488437@sss.pgh.pa.us> <001d01d953f2$099b6430$1cd22c90$@pcorp.us> <20230313115916.ivvg4tj54szeu5rj@c19> <000701d955f6$dfaaf8a0$9f00e9e0$@pcorp.us> <20230316101418.2a32erangr3nvnuk@c19> <1103862.1679352440@sss.pgh.pa.us> <007601d95c01$95744cd0$c05ce670$@pcorp.us>
Comments: In-reply-to "Regina Obe" <lr@pcorp.us>
	message dated "Tue, 21 Mar 2023 10:29:44 -0400"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <1219579.1679410331.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Tue, 21 Mar 2023 10:52:11 -0400
Message-ID: <1219580.1679410331@sss.pgh.pa.us>
Archived-At: <https://www.postgresql.org/message-id/1219580.1679410331%40sss.pgh.pa.us>
Precedence: bulk

"Regina Obe" <lr@pcorp.us> writes:
>> making the no_relocate values visible somehow, I'm not convinced that
>> pg_available_extension_versions should be the place to do it.  ISTM wha=
t's
>> relevant is the no_relocate values of *installed* extensions, not those=
 of
>> potentially-installable extensions.  If we had a view over pg_extension=
 then
>> that might be a place to add this, but we don't.  On the whole it didn'=
t seem
>> important enough to pursue, so I just left it out.

> Thanks.  Agree with get_available_versions_for_extension, not necessary.

If we did feel like doing something about this, on reflection I think
the thing to do would be to add no_relocate as an actual column in
pg_extension, probably of type "oid[]".  Then we could modify the
SET SCHEMA code to check that instead of parsing the extension control
files.  That'd be a little cleaner, but I can't say that I'm hugely
excited about it.

			regards, tom lane