From: | Japin Li <japinli(at)hotmail(dot)com> |
---|---|
To: | Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Typo in xact.c |
Date: | 2022-09-16 04:01:46 |
Message-ID: | MEYP282MB1669BE95A8E637D487E7B4F2B6489@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, 16 Sep 2022 at 11:11, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> wrote:
> At Thu, 15 Sep 2022 22:38:01 +0800, Japin Li <japinli(at)hotmail(dot)com> wrote in
>>
>> Hi hacker,
>>
>> Recently, I find there might be a typo in xact.c comments. The comments
>> say "PG_PROC", however, it actually means "PGPROC" structure. Since we
>> have pg_proc catalog, and use PG_PROC to reference the catalog [1], so,
>> we should use PGPROC to reference the structure. Any thoughts?
>>
>> [1] src/include/nodes/primnodes.h
>
> The patch seems to me covering all occurances of PG_PROC as PGPROC.
>
> I found several uses of PG_PROC as (pg_catalog.)pg_proc, which is
> quite confusing, too..
>
> regards.
Thanks for the review. Attached a new patch to replace upper case system
catatlog to lower case [1].
--
Regrads,
Japin Li.
ChengDu WenWu Information Technology Co.,Ltd.
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2022-09-16 04:08:30 | Re: missing indexes in indexlist with partitioned tables |
Previous Message | Julien Rouhaud | 2022-09-16 03:59:04 | Re: Schema variables - new implementation for Postgres 15 |