On Tue, Oct 5, 2021 at 7:44 AM Amul Sul <sulamul@gmail.com> wrote: > I was trying to understand the v1 patch and found that at the end > RequestCheckpoint() is called unconditionally, I think that should > have been called if REDO had performed,
You're right. But I don't think we need an extra variable like this, right? We can just test InRecovery?
No, InRecovery flag get cleared before this point. I think, we can use lastReplayedEndRecPtr what you have suggested in other thread.