Andres Freund <andres@2ndquadrant.com> writes:
> What I dislike with what you committed is that the state you're
> investigating during the pause isn't the one youre going to end up
> recoveryApply == true. That seems dangerous to me, even if its going to
> be reworked in HEAD.
Agreed, but it's been like that since the pause feature was created, and
we've not heard complaints (which TBH makes me wonder how much use the
feature has actually seen). I'm hesitant to change the semantics in the
back branches more than necessary to avoid clear-showstopper cases.
regards, tom lane