On Fri, Jan 11, 2019 at 11:17:48AM +0100, Peter Eisentraut wrote:
> Attached revised 0002 with those changes.
This one looks fine.
> In that test, if I change the 'current' to 'latest', the test doesn't
> fail, so it's probably not a good test.
I can see your point. You would need a diverging timeline to test for
'latest', which can surely be done as part of 003_recovery_targets.pl.
It seems to me that that the test has initial value to make sure that
we replay up to the end of the produced timeline's data, which is
something untested now as the script has only restart points set to
before the end of the timeline. If you think that's not a good
addition now, I am also fine to not include it.
--
Michael
