Hi,
On April 8, 2025 9:21:57 AM EDT, Tomas Vondra <tomas@vondra.me> wrote:
>On 4/8/25 15:06, Andres Freund wrote:
>> Hi,
>>
>> On 2025-04-08 17:44:19 +0500, Kirill Reshke wrote:
>>> On Mon, 7 Apr 2025 at 23:00, Tomas Vondra <tomas@vondra.me> wrote:
>>>> I'll let the CI run the tests on it, and
>>>> then will push, unless someone has more comments.
>>>>
>>>
>>>
>>> Hi! I noticed strange failure after this commit[0]
>>>
>>> Looks like it is related to 65c298f61fc70f2f960437c05649f71b862e2c48
>>>
>>> In file included from [01m [K../pgsql/src/include/postgres.h:49 [m [K,
>>> from [01m [K../pgsql/src/port/pg_numa.c:16 [m [K:
>>> [01m [K../pgsql/src/include/utils/elog.h:79:10: [m [K
>>> [01;31m [Kfatal error: [m [Kutils/errcodes.h: No such file or
>>> directory
>>> 79 | #include [01;31m [K"utils/errcodes.h" [m [K
>>> | [01;31m [K^~~~~~~~~~~~~~~~~~ [m [K
>>> compilation terminated.
>>
>> $ ninja -t missingdeps
>> Missing dep: src/port/libpgport.a.p/pg_numa.c.o uses src/include/utils/errcodes.h (generated by CUSTOM_COMMAND)
>> Missing dep: src/port/libpgport_shlib.a.p/pg_numa.c.o uses src/include/utils/errcodes.h (generated by
CUSTOM_COMMAND)
>> Processed 2384 nodes.
>> Error: There are 2 missing dependency paths.
>> 2 targets had depfile dependencies on 1 distinct generated inputs (from 1 rules) without a non-depfile dep path to
thegenerator.
>> There might be build flakiness if any of the targets listed above are built alone, or not late enough, in a clean
outputdirectory.
>>
>>
>> I think it's not right that something in src/port defines an SQL callable
>> function. The set of headers for that pull in a lot of things.
>>
>> Since the file relies on things like GUCs, I suspect this should be in
>> src/backend/port or such instead.
>>
>
>Yeah, I think you're right, src/backend/port seems like a better place
>for this. I'll look into moving that in the evening.
On a second look I wonder if just the SQL function and perhaps the page size function should be moved. There are FE
programsthat could potentially benefit from num a awareness (e.g. pgbench).
Andres
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
