On Fri, May 15, 2015 at 10:49:43AM -0400, Stephen Frost wrote:
> Bruce,
> 
> * Bruce Momjian (br...@momjian.us) 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.

OK, thanks.  I will apply it all branches later today as it is a data
loss bug.

-- 
  Bruce Momjian  <br...@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + Everyone has their own god. +


-- 
Sent via pgsql-general mailing list (pgsql-general@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-general

Reply via email to