On 6/11/16 8:22 AM, Michael Paquier wrote: > On Sat, Jun 11, 2016 at 9:44 AM, Thom Brown <t...@linux.com> wrote: >> It may be the wrong way of going about it, but you get the idea of what I'm >> suggesting we output instead. > > Surely things could be better. So +1 to be more verbose here. > > + if (recoveryStopTime == 0) > + snprintf(reason, sizeof(reason), > + "recovery reached consistency before recovery > target time of \"%s\"\n", > + timestamptz_to_str(recoveryTargetTime)); > "Reaching consistency" is not exact for here. I'd rather say "finished > recovery without reaching target blah" > > + if (recoveryStopXid == 0) > Checking for InvalidTransactionId is better here. > > And it would be good to initialize recoveryStopTime and > recoveryStopXid as those are set only when a recovery target is > reached.
+1 for Michael's wording. It's not very clear in the logs right now if a recovery target was missed. That makes it very difficult for the user to determine if PITR worked or not. -- -David da...@pgmasters.net -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers