Hi,
On 2021-08-07 12:01:31 -0700, Andres Freund wrote:
> Attached is a patch showing how this could look like. Note that the PANIC
> should likely not be that but a WARNING, but the PANIC more useful for running
> some initial tests...
I pushed a slightly evolved version of this. As the commit message noted, this
may not be the best approach, but we can revise after further discussion.
> I'm not sure whether we'd want to continue having the proc exit hook? It seems
> to me that asserts would provide a decent enough protection against
> introducing new temp files during shutdown.
> Alternatively we could make the asserts in OpenTemporaryFile et al
> elog(ERROR)s, and be pretty certain that no temp files would be open too late?
I ended up removing the proc exit hook and not converting the asserts to an
elog(). Happy to change either.
Greetings,
Andres Freund