Re: Latest version of Hot Standby patch - Mailing list pgsql-hackers

From Simon Riggs
Subject Re: Latest version of Hot Standby patch
Date
Msg-id 1231366088.18005.90.camel@ebony.2ndQuadrant
Whole thread Raw
In response to Re: Latest version of Hot Standby patch  (Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>)
Responses Re: Latest version of Hot Standby patch  (Simon Riggs <simon@2ndQuadrant.com>)
List pgsql-hackers
On Wed, 2009-01-07 at 23:56 +0200, Heikki Linnakangas wrote:
> Simon Riggs wrote:
> > On Wed, 2009-01-07 at 15:43 +0200, Heikki Linnakangas wrote:
> >> When there's no xids in the procarray, couldn't we just use 
> >> latestCompletedXid instead of calling ReadNewTransactionId()?
> > 
> > latestCompletedXid is protected by ProcArrayLock so not much difference
> > between those two.
> 
> The big difference is that we're already holding ProcArrayLock. You 
> could read the value of latestCompletedXid before releasing 
> ProcArrayLock, and wouldn't need the retry logic.

Sounds good to me then. Will rework.

-- Simon Riggs           www.2ndQuadrant.comPostgreSQL Training, Services and Support



pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: Significant oversight in that #include-removal script
Next
From: Stephen Frost
Date:
Subject: Re: Do we still need constraint_exclusion?