Bruce, * Bruce Momjian (bruce@xxxxxxxxxx) wrote: > On Mon, Mar 9, 2015 at 12:43:05PM -0400, Bruce Momjian wrote: > > On Fri, Mar 6, 2015 at 06:10:15PM -0500, Stephen Frost wrote: > > > The first is required or anyone who has done that will get the funny > > > error that started this thread and things won't work anyway, but I > > > believe the latter is also necessary to patch and back-patch as it could > > > lead to data loss. It's not a high potential as, hopefully, people will > > > check first, but I can imagine a hosting provider or environments where > > > there are lots of independent clusters not catching this issue in their > > > testing, only to discover someone set their database to 'datallowconn = > > > false' for whatever reason and now that database is gone... > > > > Agreed. I will work on a patch for this. > > Attached is a patch that implements this, and it should be backpatch to > all versions. Excellent and agreed. Just looked through the patch and didn't do a full review, but it looks good to me. Thanks! Stephen
Attachment:
signature.asc
Description: Digital signature