Re: Isn't init_irels() dangerous ? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Isn't init_irels() dangerous ?
Date
Msg-id 22103.977531860@sss.pgh.pa.us
Whole thread Raw
In response to Re: Isn't init_irels() dangerous ?  (Hiroshi Inoue <Inoue@tpf.co.jp>)
List pgsql-hackers
Hiroshi Inoue <Inoue@tpf.co.jp> writes:
> Tom Lane wrote:
>> "Hiroshi Inoue" <Inoue@tpf.co.jp> writes:
>>>> It seems that init_irels() should be called after
>>>> InitializeTransactionSystem() was called.
>> 
>> Can we just swap the order of the RelationCacheInitialize() and
>> InitializeTransactionSystem() calls in InitPostgres?  If that
>> works, I'd have no objection.

> It doesn't work. InitializeTransactionSystem() requires
> pg_log/pg_variable relations which are already built in 
> RelationCacheInitialize().

OK.  Second proposal: do the init_irels() call in
RelationCacheInitializePhase2().  I've just looked through the
other stuff that's done in between, and I don't think any of it
needs valid relcache entries.

> In the meantime,I have another anxiety. init_irels()
> (RelationCacheInitialize()) seems to be called while 
> Locking is disabled.

This should fix that problem, too.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: GEQO status?
Next
From: Tom Lane
Date:
Subject: Re: GEQO status?