On Thu, Sep 27, 2018 at 1:32 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Thu, Sep 27, 2018 at 11:18:02AM +0530, Amit Kapila wrote:
> > Your proposed solution makes sense to me. IIUC, this is quite similar
> > to what Dilip has also proposed [1].
>
> Indeed. I would just add with the patch a comment like that:
> "Perform this call outside the critical section so as if the instance
> just got out of recovery, the upcoming WAL insert initialization does
> not trigger an assertion failure."
>
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?
> Sure, feel free to if you have some room. I am fine to take care of it
> as well, so that's up to you to decide.
Okay, I will take care of it.
> Adding a comment like what I
> proposed upthread is necessary in my opinion.
Agreed.
--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com