On 2023-08-01 12:23, Andres Freund wrote:
> Hi,
>
> On 2023-08-01 12:14:49 +0900, Michael Paquier wrote:
>> On Tue, Aug 01, 2023 at 11:51:35AM +0900, Masahiro Ikeda wrote:
>> > Thanks for committing the main patch.
>> >
>> > In my understanding, the rest works are
>> > * to support WaitEventExtensionMultiple()
>> > * to replace WAIT_EVENT_EXTENSION to custom wait events
>> >
>> > Do someone already works for them? If not, I'll consider
>> > how to realize them.
>>
>> Note that postgres_fdw and dblink use WAIT_EVENT_EXTENSION, but have
>> no dependency to shared_preload_libraries. Perhaps these could just
>> use a dynamic handling but that deserves a separate discussion because
>> of the fact that they'd need shared memory without being able to
>> request it. autoprewarm.c is much simpler.
>
> This is why the scheme as implemented doesn't really make sense to me.
> It'd be
> much easier to use if we had a shared hashtable with the identifiers
> than
> what's been merged now.
>
> In plenty of cases it's not realistic for an extension library to run
> in each
> backend, while still needing to wait for things.
OK, I'll try to make a PoC patch.
Regards,
--
Masahiro Ikeda
NTT DATA CORPORATION