From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | John Naylor <johncnaylorls(at)gmail(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: generate syscache info automatically |
Date: | 2024-01-17 12:46:02 |
Message-ID: | 4a72ce01-87c2-41a6-baab-f44a4b39e664@eisentraut.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 10.01.24 09:00, John Naylor wrote:
>> I suppose a sensible alternative could be to leave the
>> DECLARE_..._INDEX... alone and make a separate statement, like
>>
>> MAKE_SYSCACHE(pg_type_oid_index, TYPEOID, 64);
>>
>> That's at least visually easier, because some of those
>> DECLARE_... lines are getting pretty long.
>
> Probably a good idea, and below I mention a third possible macro.
I updated the patch to use this style (but I swapped the first two
arguments from my example, so that the thing being created is named first).
I also changed the names of the output files a bit to make them less
confusing. (I initially had some files named .c.h, which was weird, but
apparently necessary to avoid confusing the build system. But it's all
clearer now.)
Other than bugs and perhaps style opinions, I think the first patch is
pretty good now.
I haven't changed much in the second patch, other than to update it for
the code changes made in the first patch. It's still very much
WIP/preview at the moment.
Attachment | Content-Type | Size |
---|---|---|
v5-0001-Generate-syscache-info-from-catalog-files.patch | text/plain | 64.4 KB |
v5-0002-WIP-Generate-ObjectProperty-from-catalog-files.patch | text/plain | 14.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2024-01-17 12:53:59 | Re: Fix a possible socket leak at Windows (src/backend/port/win32/socket.c) |
Previous Message | Nazir Bilal Yavuz | 2024-01-17 12:20:39 | Re: Show WAL write and fsync stats in pg_stat_io |