Re: ResourceOwner refactoring

From: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
To: Aleksander Alekseev <aleksander(at)timescale(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Cc: Andres Freund <andres(at)anarazel(dot)de>, Julien Rouhaud <rjuju123(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>
Subject: Re: ResourceOwner refactoring
Date: 2023-05-25 17:14:23
Message-ID: 527af829-4a03-03b0-121c-46ff79c61887@iki.fi
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 23/05/2023 16:41, Aleksander Alekseev wrote:
> Hi,
>
>> [...]
>> A-ha, it's because I didn't add the new test_resowner directory to the
>> src/test/modules/meson.build file. Fixed.
>>
>> Thanks for the review!
>
> You probably already noticed, but for the record: cfbot seems to be
> extremely unhappy with the patch.

Thanks, here's a fixed version. (ResourceOwner resource release
callbacks mustn't call ResourceOwnerForget anymore, but AbortBufferIO
was still doing that)

--
Heikki Linnakangas
Neon (https://neon.tech)

Attachment Content-Type Size
v15-0001-Move-a-few-ResourceOwnerEnlarge-calls-for-safety.patch text/x-patch 9.8 KB
v15-0002-Make-resowners-more-easily-extensible.patch text/x-patch 151.9 KB
v15-0003-Use-a-faster-hash-function-in-resource-owners.patch text/x-patch 2.6 KB
v15-0004-Change-pgcrypto-to-use-the-new-ResourceOwner-mec.patch text/x-patch 7.9 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephen Frost 2023-05-25 17:29:12 Re: Docs: Encourage strong server verification with SCRAM
Previous Message Stephen Frost 2023-05-25 17:08:40 Re: Large files for relations