Re: Transaction timeout

x4mmm@yandex-team.ru

From: "Andrey M. Borodin" <x4mmm@yandex-team.ru>

To: Peter Smith <smithpb2250@gmail.com>

Cc: Japin Li <japinli@hotmail.com>, Junwang Zhao <zhjwpku@gmail.com>, 邱宇航 <iamqyh@gmail.com>, Fujii Masao <masao.fujii@oss.nttdata.com>, Andrey Borodin <amborodin86@gmail.com>, Andres Freund <andres@anarazel.de>, Michael Paquier <michael.paquier@gmail.com>, Nikolay Samokhvalov <samokhvalov@gmail.com>, pgsql-hackers <pgsql-hackers@postgresql.org>, "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>

Date: 2024-01-26T06:46:43Z

Lists: pgsql-hackers

Attachments

v24-0001-Introduce-transaction_timeout.patch (application/octet-stream) patch v24-0001


> On 26 Jan 2024, at 11:44, Andrey M. Borodin <x4mmm@yandex-team.ru> wrote:
> 
> 
> 1. It’s unsafe for isaoltion tester to await transaction_timeout within a query. Usually it gets
> FATAL: terminating connection due to transaction timeout
> But if VM is a bit slow it can get occasional
> PQconsumeInput failed: server closed the connection unexpectedly
> So, currently all tests use “passive waiting”, in a session that will not timeout.


Oops, sorry, I’ve accidentally sent version without this fix.
Here it is.


Best regards, Andrey Borodin.

Commits

Add TAP tests for timeouts
- eeefd4280f6e 17.0 landed
Remove flaky isolation tests for timeouts
- a661bf7b0f56 17.0 landed
Followup fixes for transaction_timeout
- bf82f43790a6 17.0 landed
Introduce transaction_timeout
- 51efe38cb92f 17.0 landed
On systems that have setsid(2) (which should be just about everything except
- 3ad0728c817b 8.2.0 cited