Re: [PATCHES] Infrastructure changes for recovery (v8) - Mailing list pgsql-hackers

From Simon Riggs
Subject Re: [PATCHES] Infrastructure changes for recovery (v8)
Date
Msg-id 1226935993.3790.12.camel@ebony.2ndQuadrant
Whole thread Raw
In response to Re: [PATCHES] Infrastructure changes for recovery (v8)  (Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>)
List pgsql-hackers
On Mon, 2008-11-17 at 16:18 +0200, Heikki Linnakangas wrote:

> > +
> > +                       /*
> > +                        * Load the flat authorization file into
> postmaster's ca
> > +                        * startup process won't have recomputed
> this from the d
> > +                        * yet, so it may change following recovery.
> > +                        */
> > +                       load_role();
> 
> Is there a race condition here too, if the startup process is writing 
> the auth file at the same time? I guess we'd have the same problem
> with flat files in general, so maybe there's something else that
> mitigates the problem?

The flat file is not race condition proof. When the file is read it is
just a guide and the real data is re-accessed from catalog. So the
problem you see does exist, but is handled elsewhere - not in this
patch.

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



pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: [PATCHES] Infrastructure changes for recovery (v8)
Next
From: Aidan Van Dyk
Date:
Subject: Re: Block-level CRC checks