Re: Transaction timeout

From: "Andrey M(dot) Borodin" <x4mmm(at)yandex-team(dot)ru>
To: Junwang Zhao <zhjwpku(at)gmail(dot)com>
Cc: Li Japin <japinli(at)hotmail(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: 2023-12-29 10:00:12
Message-ID: 68965408-F5A3-459F-A84E-5F4F2C237A5B@yandex-team.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 28 Dec 2023, at 21:02, Junwang Zhao <zhjwpku(at)gmail(dot)com> wrote:
>
> Seems V5~V17 doesn't work as expected for Nikolay's case:
>

Yeah, that's a problem.
> So I propose the following change, what do you think?
This breaks COMMIT AND CHAIN.

PFA v18: I've added a test for Nik's case and for COMMIT AND CHAIN. Now we need to fix stuff to pass this tests (I've crafted output).
We also need test for patchset step "Try to enable transaction_timeout before next command".

Thanks!

Best regards, Andrey Borodin.

Attachment Content-Type Size
v18-0001-Introduce-transaction_timeout.patch application/octet-stream 19.8 KB
v18-0002-Use-test-from-Li-Japin.patch application/octet-stream 9.0 KB
v18-0003-Try-to-enable-transaction_timeout-before-next-co.patch application/octet-stream 1.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrey M. Borodin 2023-12-29 10:02:48 Re: libpq compression (part 3)
Previous Message Andrei Lepikhov 2023-12-29 09:58:11 Re: Removing unneeded self joins