Re: Verified fix for Bug 4137 - Mailing list pgsql-patches

From Heikki Linnakangas
Subject Re: Verified fix for Bug 4137
Date
Msg-id 4824604F.5080707@enterprisedb.com
Whole thread Raw
In response to Re: Verified fix for Bug 4137  (Simon Riggs <simon@2ndquadrant.com>)
Responses Re: Verified fix for Bug 4137  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-patches
Simon Riggs wrote:
> I've extended the patch without introducing another new status variable,
> which was my original concern with what you suggested previously.

Ok, that'll work. Committed, thanks. I changed the sanity check that
xlogfname > restore point into an Assert, though, because that's a sign
that something's wrong.

I noted that there's no reason why the truncation point calculation had
to be moved outside the case-statement, but it does look better that way
so I did apply that change.

--
   Heikki Linnakangas
   EnterpriseDB   http://www.enterprisedb.com

pgsql-patches by date:

Previous
From: Peter Eisentraut
Date:
Subject: New flex warnings
Next
From: "Heikki Linnakangas"
Date:
Subject: Re: Verified fix for Bug 4137