On Wed, Jun 20, 2018 at 08:43:11PM -0700, Andres Freund wrote: > On 2018-06-18 11:13:47 -0700, Andres Freund wrote: >> We could do that - but add_to_unowned_list() is actually a bottleneck in >> other places during recovery too. We pretty much never (outside of >> dropping relations / databases) close opened relations during recovery - >> which is obviously problematic since nearly all of the entries are >> unowned. I've come to the conclusion that we should have a global >> variable that just disables adding anything to the global lists. > > On second thought: I think we should your approach in the back branches, > and something like I'm suggesting in master once open.
+1. Let's also make sure that the removal of smgrdounlink and smgrdounlinkfork happens only on master. -- Michael
signature.asc
Description: PGP signature