Dear Alexander,
> Regarding last_saved_restart_lsn_updated, I think the opposite. I
> think we should check if last_saved_restart_lsn_updated is set already
> only if it could promise us some economy of resources. In our case
> the main check only compares two fields of slot. And that fields are
> to be accessed anyway. So, we are not going to save any RAM accesses.
> Therefore, checking for last_saved_restart_lsn_updated seems like
> unnecessary code complication (and I don't see we're doing that in
> other places). So, I'm going to push this patch "as is".
To clarify: I have no objections. Thanks for giving the knowledge.
Best regards,
Hayato Kuroda
FUJITSU LIMITED