Re: Add checkpoint and redo LSN to LogCheckpointEnd log message

On Tue, Feb 1, 2022 at 11:58 AM Kyotaro Horiguchi
<horikyota(dot)ntt(at)gmail(dot)com> wrote:
> > Modified in v8.
>
> 0001 looks good to me.
>
> I tend to agree to 0002.

Thanks.

> FWIW, I collected other user-facing usage of "location" as LSN.
>
> xlog.c:11298, 11300: (in backup-label)
> appendStringInfo(labelfile, "START WAL LOCATION: %X/%X (file %s)\n",
> appendStringInfo(labelfile, "CHECKPOINT LOCATION: %X/%X\n",
> (and corresponding reader-code)
>
> xlog,c:11791, 11793: (in backup history file)
> fprintf(fp, "START WAL LOCATION: %X/%X (file %s)\n",
> fprintf(fp, "STOP WAL LOCATION: %X/%X (file %s)\n",
> (and corresponding reader-code)

I tried to change the "location" to "lsn" in most of the user-facing
messages/text. I refrained from changing the bakup_label file content
(above) as it might break many applications/service layer code and
it's not good for backward compatibility.

Attaching the above changes 0003 (0001 and 0002 remain the same). If
the committer doesn't agree on the text or wording in 0003, I would
like the 0001 and 0002 to be taken here and I can start a new thread
for discussing 0003 separately.

Regards,
Bharath Rupireddy.