Re: Weird test mixup - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Weird test mixup
Date
Msg-id ZffblQnhTivQH9q6@paquier.xyz
Whole thread Raw
In response to Re: Weird test mixup  ("Andrey M. Borodin" <x4mmm@yandex-team.ru>)
List pgsql-hackers
On Mon, Mar 18, 2024 at 10:50:25AM +0500, Andrey M. Borodin wrote:
> Maybe consider function injection_points_attach_local(‘point name’)
> instead of static switch?
> Or even injection_points_attach_global(‘point name’), while function
> injection_points_attach(‘point name’) will be global? This would
> favour writing concurrent test by default…

The point is to limit accidents like the one of this thread.  So, for
cases already in the tree, not giving the point name in the SQL
function would be simple enough.

What you are suggesting can be simply done, as well, though I'd rather
wait for a reason to justify doing so.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Switching XLog source from archive to streaming when primary available
Next
From: Peter Eisentraut
Date:
Subject: Re: What about Perl autodie?