Re: pgsql: Restrict the use of temporary namespace in two-phase transaction - Mailing list pgsql-hackers

From Tom Lane
Subject Re: pgsql: Restrict the use of temporary namespace in two-phase transaction
Date
Msg-id 499.1547843670@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Restrict the use of temporary namespace in two-phase transaction  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: pgsql: Restrict the use of temporary namespace in two-phasetransaction  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
Robert Haas <robertmhaas@gmail.com> writes:
> On Thu, Jan 17, 2019 at 8:08 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>>> Anyway, it seems to me that this is pointing out to another issue:
>>> current_schema() can trigger a namespace creation, hence shouldn't we
>>> mark it as PARALLEL UNSAFE and make sure that we never run into this
>>> problem?

>> That seems a bit annoying, but maybe we have little choice?

> The only other option I see is to make current_schema() not trigger a
> namespace creation.

Seems hard to avoid.  We could conceivably make it return "pg_temp"
for the temp schema instead of the schema's actual name, but it's
not very hard to think of ways whereby that would make use of the
result fail in contexts where it previously worked.

Another idea is to force creation of the temp namespace as soon as
we see that search_path references it.  I'm not very sure exactly
where would be a convenient place to make that happen, though.
There are paths whereby the GUC's value will change outside a
transaction, so we couldn't tie it directly to the GUC update.

            regards, tom lane


pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Shouldn't current_schema() be at least PARALLEL RESTRICTED?
Next
From: Robert Haas
Date:
Subject: Re: Early WIP/PoC for inlining CTEs