Re: Verified fix for Bug 4137

From: "Heikki Linnakangas" <heikki(at)enterprisedb(dot)com>
To: "Simon Riggs" <simon(at)2ndquadrant(dot)com>
Cc: <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Verified fix for Bug 4137
Date: 2008-05-09 14:31:43
Message-ID: 4824604F.5080707@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Heikki Linnakangas 2008-05-09 14:37:12 Re: Verified fix for Bug 4137
Previous Message Tom Lane 2008-05-09 14:23:12 Re: [0/4] Proposal of SE-PostgreSQL patches