Re: Weird test mixup - Mailing list pgsql-hackers

From Andrey M. Borodin
Subject Re: Weird test mixup
Date
Msg-id FB71B3B1-D68F-4508-BA95-D427792FAB2B@yandex-team.ru
Whole thread Raw
In response to Re: Weird test mixup  (Michael Paquier <michael@paquier.xyz>)
Responses Re: Weird test mixup
List pgsql-hackers

> On 5 Apr 2024, at 07:19, Michael Paquier <michael@paquier.xyz> wrote:
>
> It's been a couple of weeks since this has been sent, and this did not
> get any reviews.  I'd still be happy with the simplicity of a single
> injection_points_local() that can be used to link all the injection
> points created in a single process to it, discarding them once the
> process exists with a shmem exit callback.

OK, makes sense.
I find name of the function "injection_points_local()" strange, because there is no verb in the name. How about
"injection_points_set_local"?

>  And I don't really see an
> argument to tweak the backend-side routines, as well.
>  Comments and/or
> objections?

I'm not sure if we should refactor anything here, but InjectionPointSharedState has singular name, plural wait_counts
andsingular condition. 
InjectionPointSharedState is already an array of injection points, maybe let's add there optional pid instead of
inventingseparate array of pids? 

Can we set global point to 'notice', but same local to 'wait'? Looks like now we can't, but allowing to do so would
makecode simpler. 

Besides this opportunity to simplify stuff, both patches looks good to me.


Best regards, Andrey Borodin.


pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Synchronizing slots from primary to standby
Next
From: Peter Eisentraut
Date:
Subject: Re: Cutting support for OpenSSL 1.0.1 and 1.0.2 in 17~?