| From: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> |
|---|---|
| To: | Dewei Dai <daidewei1970(at)163(dot)com> |
| Cc: | Pgsql Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Fix typo in code comment |
| Date: | 2026-01-08 15:24:23 |
| Message-ID: | CAExHW5vT3h3F6S__VdeSO797uE0XzocOEw_r4PKDgF5gxr3Pcw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Jan 7, 2026 at 4:50 PM Dewei Dai <daidewei1970(at)163(dot)com> wrote:
>
> Hi hickers,
> I fixed a comment error in origin.c.
> The original comment was "* Replication origin consist out of a descriptive, user defined, externa..."
> where I think "consist out of" is a typo and it should be "consists of".
>
You are right. LGTM.
However, I find use of "consist of" itself not so accurate. consists
of is used to denote relationship between whole and its parts. But
what the sentence talks about is alternate representations of
replication origin - long and short one. I think the sentence should
be rewritten as "Replication origin has two interchangeable
identifications: a descriptive .... ". What do you think?
--
Best Wishes,
Ashutosh Bapat
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ashutosh Bapat | 2026-01-08 15:25:32 | Re: Refactor replication origin state reset helpers |
| Previous Message | Bruce Momjian | 2026-01-08 15:03:22 | Re: [PATCH] Provide support for trailing commas |