On Thu, Sep 27, 2018 at 6:22 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Thu, Sep 27, 2018 at 04:19:02PM +0530, Amit Kapila wrote:
> > I think this is mostly fine, but it seems "if the instance just got
> > out of recovery" doesn't fit well because it can happen anytime after
> > recovery, this code gets called from checkpointer. I think we can
> > slightly tweak it as below:
> > "Perform this outside critical section so that the WAL insert
> > initialization done by RecoveryInProgress() doesn't trigger an
> > assertion failure."
> >
> > What do you say?
>
> Fine for me.
>
Okay, I am planning to commit the attached patch tomorrow unless you
or anybody else has any objections to it.
--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com