Re: [PATCHES] Infrastructure changes for recovery (v8)

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