Re: Transaction timeout

From: "Andrey M(dot) Borodin" <x4mmm(at)yandex-team(dot)ru>
To: Japin Li <japinli(at)hotmail(dot)com>
Cc: Junwang Zhao <zhjwpku(at)gmail(dot)com>, 邱宇航 <iamqyh(at)gmail(dot)com>, Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>, Andrey Borodin <amborodin86(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Nikolay Samokhvalov <samokhvalov(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Transaction timeout
Date: 2024-01-04 08:41:43
Message-ID: DD2C0791-FCDB-4073-848C-7107CAB55EAE@yandex-team.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 4 Jan 2024, at 07:14, Japin Li <japinli(at)hotmail(dot)com> wrote:
>
> Does the timeout is too short for testing? I see the timeouts for lock_timeout
> and statement_timeout is more bigger than transaction_timeout.

Makes sense. Done. I've also put some effort into fine-tuning timeouts Nik's case tests. To have 100ms gap between check, false positive and actual bug we had I had to use transaction_timeout = 300ms. Currently all tests take more than 1000ms!
But I do not see a way to make these tests both stable and short.

Best regards, Andrey Borodin.

Attachment Content-Type Size
v22-0001-Introduce-transaction_timeout.patch application/octet-stream 15.2 KB
v22-0002-Add-tests-for-transaction_timeout.patch application/octet-stream 9.2 KB
v22-0003-Try-to-enable-transaction_timeout-before-next-co.patch application/octet-stream 1.4 KB
v22-0004-fix-reschedule-timeout-for-each-commmand.patch application/octet-stream 1.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message vignesh C 2024-01-04 08:51:51 Documentation to upgrade logical replication cluster
Previous Message Michael Banck 2024-01-04 07:30:36 Re: [PATCH] Exponential backoff for auth_delay