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
