Hi,

I think the changes make sense. Would it be helpful to add the origin of the checkpoint record we are referring to ? (i.e. control file or backup label).

For example:

* Latest checkpoint in the control file is at %X/%X on timeline %u,
* Checkpoint location in the backup_label file is at %X/%X on timeline %u,

The current message refers to "Latest checkpoint" which automatically makes me think about the control file (specifically, the pg_controldata output).

Now that I have read the code, I understand which one we are referring to, but for someone who hasn't (or me in a few month) it might be useful ?

--
Benoit Lobréau
Consultant
http://dalibo.com



Reply via email to