On Fri, Jan 5, 2018 at 11:27 PM, Simon Riggs <si...@2ndquadrant.com> wrote: > I'm not. > > If we want to do this why not only do it in the modes that have meaning? > i.e. put an if() test in for archive_mode == always.
OK, I can see value in your point as well. The check is a bit more complicated that just looking for archive_mode == always though, you need to look at if the backup has been started in recovery or not, and then adapt using XLogArchivingActive() and XLogArchivingAlways(), similarly to what is done when waiting for the archives. > Which also makes it a smaller and clearer patch Yes, this generates less diffs, reducing the likelihood of bugs. What do you think about the v3 attached? -- Michael
standby-backup-history-v3.patch
Description: Binary data
