Hello
Thank you! I'm on vacation, so I was finally able to review the patch.
Seems WAIT_EVENT_RECOVERY_PAUSE addition was lost during patch simplification.
> ereport(FATAL,
> (errmsg("recovery aborted because of insufficient parameter settings"),
> errhint("You can restart the server after making the necessary configuration changes.")));
I think we should repeat here conflicted param_name and minValue. pg_wal_replay_resume can be called days after
recoverybeing paused. The initial message can be difficult to find.
> errmsg("recovery will be paused")
May be use the same "recovery has paused" as in recoveryPausesHere? It doesn't seem to make any difference since we set
pauseright after that, but there will be a little less work translators.
Not sure about "If recovery is unpaused". The word "resumed" seems to have been usually used in docs.
regards, Sergei