From: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Minor refactorings to eliminate some static buffers |
Date: | 2024-07-30 19:24:57 |
Message-ID: | 3b4e725d-710e-4d36-a8f8-b13e8ece3ba1@iki.fi |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 30/07/2024 18:44, Robert Haas wrote:
> On Tue, Jul 30, 2024 at 7:22 AM Heikki Linnakangas <hlinnaka(at)iki(dot)fi> wrote:
>> As part of the multithreading work, it'd be nice to get rid of as many
>> global or static variables as possible. Remaining ones can be converted
>> to thread locals as appropriate, but where possible, it's better to just
>> get rid of them.
>>
>> Here are patches to get rid of a few static variables, by e.g.
>> converting them to regular local variables or palloc'd return values, as
>> appropriate.
>>
>> This doesn't move the needle much, but every little helps, and these
>> seem like nice little changes in any case.
>
> I spent a few minutes looking through these patches and they seem like
> good cleanups. I couldn't think of a plausible reason why someone
> would object to any of these.
Committed, thanks for having a look.
--
Heikki Linnakangas
Neon (https://neon.tech)
From | Date | Subject | |
---|---|---|---|
Next Message | Jeff Davis | 2024-07-30 19:31:24 | [17+] check after second call to pg_strnxfrm is too strict, relax it |
Previous Message | Jeff Davis | 2024-07-30 19:13:12 | Re: tiny step toward threading: reduce dependence on setlocale() |