On 4/5/22 11:25 AM, Stephen Frost wrote:
>
> Please find attached an updated patch + commit message. Mostly, I just
> went through and did a bit more in terms of updating the documentation
> and improving the comments (there were some places that were still
> worrying about the chance of a 'stray' backup_label file existing, which
> isn't possible any longer), along with some additional testing and
> review. This is looking pretty good to me, but other thoughts are
> certainly welcome. Otherwise, I'm hoping to commit this tomorrow.
I have reviewed the changes and they look good. I also ran the new patch
through pgbackrest regression with no issues.
Regards,
--
-David
david@pgmasters.net