From: "Regina Obe" <lr@pcorp.us>
To: "'Tom Lane'" <tgl@sss.pgh.pa.us>,
	"'Sandro Santilli'" <strk@kbt.io>
Cc: "'Gregory Stark \(as CFM\)'" <stark.cfm@gmail.com>,
	<pgsql-hackers@lists.postgresql.org>,
	"'Regina Obe'" <r@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>
In-Reply-To: <1103862.1679352440@sss.pgh.pa.us>
Subject: RE: Ability to reference other extensions by schema in extension scripts
Date: Tue, 21 Mar 2023 10:29:44 -0400
Message-ID: <007601d95c01$95744cd0$c05ce670$@pcorp.us>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: 7bit
Thread-Index: AQF1aS96Q/mFBruZUMXyXpCm6Y8ZOgLXfdbzAh2KTDUCqQCSKwHAG++5AmsEVhwCGfZasQMhUTeiAmG7vz8Bz1Ba7gJSqW73AyS1Kleu98iWIA==
Content-Language: en-us
Archived-At: <https://www.postgresql.org/message-id/007601d95c01%2495744cd0%24c05ce670%24%40pcorp.us>
Precedence: bulk

> Pushed with some mostly-cosmetic adjustments (in particular I tried to
make
> the docs and tests neater).
> 
> I did not commit the changes in get_available_versions_for_extension
> to add no_relocate as an output column.  Those were dead code because you
> hadn't done anything to connect them up to an actual output parameter of
> pg_available_extension_versions().  While I'm not necessarily averse to
> making the no_relocate values visible somehow, I'm not convinced that
> pg_available_extension_versions should be the place to do it.  ISTM what'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.
> 
> 			regards, tom lane


Thanks.  Agree with get_available_versions_for_extension, not necessary.

Thanks,
Regina