Re: Add isolation test template in injection_points for wait/wakeup/detach

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com>
Cc: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Add isolation test template in injection_points for wait/wakeup/detach
Date: 2024-10-29 08:20:42
Message-ID: ZyCa2qsopKaw3W3K@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Oct 29, 2024 at 07:59:31AM +0000, Bertrand Drouvot wrote:
> On Tue, Oct 29, 2024 at 01:12:25PM +0900, Michael Paquier wrote:
>> Under a wait of N seconds,
>> we could still have environments where the wait() could remain stuck
>> more than N seconds between the moment the condition variable is woken
>> up and the result of the wait() is reported back to the client. And
>> hardcoded sleeps make the test slower even on fast machines.
>
> Yeah, not ideal so better to keep the test as it is currently (after your
> refactoring).

Backpedalling a bit on that.. Perhaps it would be useful to keep the
permutation as disabled in the test and to update the comment in
basic.spec to tell that this had better be avoided when implementing a
test if the timing of the return value of a SQL wait() cannot be
strictly controlled? And also say that putting the wakeup() should be
last for simple permutations? The point of this spec is also to
document some good and bad practices.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message jian he 2024-10-29 08:28:46 Re: general purpose array_sort
Previous Message Арсений Косицын 2024-10-29 08:08:37 [PATCH] Fixed creation of empty .log files during log rotation