From: | Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> |
---|---|
To: | tgl(at)sss(dot)pgh(dot)pa(dot)us |
Cc: | masao(dot)fujii(at)oss(dot)nttdata(dot)com, bharath(dot)rupireddyforpostgres(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Remove useless arguments in ReadCheckpointRecord(). |
Date: | 2022-07-22 08:17:12 |
Message-ID: | 20220722.171712.22612542004155123.horikyota.ntt@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
At Thu, 21 Jul 2022 23:10:04 -0400, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote in
> Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> writes:
> > On 2022/07/21 14:54, Kyotaro Horiguchi wrote:
> >> At Thu, 21 Jul 2022 11:45:23 +0900, Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> wrote in
> >>> - (errmsg("could not locate required checkpoint record"),
> >>> + (errmsg("could not locate a valid checkpoint record in backup_label file"),
>
> >> "in backup_label" there looks *to me* need some verb..
>
> > Sorry, I failed to understand your point. Could you clarify your point?
>
> FWIW, the proposed change looks like perfectly good English to me.
> "locate" is the verb. It's been way too many years since high
> school grammar for me to remember the exact term for auxiliary
> clauses like "in backup_label file", but that doesn't need its
> own verb. Possibly Kyotaro-san is feeling that it should be
> like "... checkpoint record in the backup_label file". That'd
> be more formal, but in the telegraphic style that we prefer for
> primary error messages, omitting the "the" is fine.
Maybe a little different. I thought that a checkpoint record cannot
be located in backup_label file. In other words what is in
backup_label file is a pointer to the record and the record is in a
WAL file.
I'm fine with the proposed sentsnse if the it makes the correct
sense. (And sorry for the noise)
By the way, I learned that the style is called "telegraphic style".
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2022-07-22 08:18:58 | Re: standby recovery fails (tablespace related) (tentative patch and discussion) |
Previous Message | Etsuro Fujita | 2022-07-22 08:14:00 | Re: Fast COPY FROM based on batch insert |