From: | Japin Li <japinli(at)hotmail(dot)com> |
---|---|
To: | "Andrey M(dot) Borodin" <x4mmm(at)yandex-team(dot)ru> |
Cc: | 邱宇航 <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 |
Subject: | Re: Transaction timeout |
Date: | 2023-12-18 09:32:22 |
Message-ID: | MEYP282MB316457B6810B840A012649AFB690A@MEYP282MB3164.AUSP282.PROD.OUTLOOK.COM |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, 18 Dec 2023 at 13:49, Andrey M. Borodin <x4mmm(at)yandex-team(dot)ru> wrote:
>> On 16 Dec 2023, at 05:58, Japin Li <japinli(at)hotmail(dot)com> wrote:
>>
>>
>> On Fri, 15 Dec 2023 at 17:51, Andrey M. Borodin <x4mmm(at)yandex-team(dot)ru> wrote:
>>>> On 8 Dec 2023, at 15:29, Japin Li <japinli(at)hotmail(dot)com> wrote:
>>>>
>>>> Thanks for updating the patch. LGTM.
>>>
>>> PFA v9. Changes:
>>> 1. Added tests for idle_in_transaction_timeout
>>> 2. Suppress statement_timeout if it’s shorter than transaction_timeout
>>>
>> + if (StatementTimeout > 0
>> + && IdleInTransactionSessionTimeout < TransactionTimeout)
>> ^
>>
>> Should be StatementTimeout?
> Yes, that’s an oversight. I’ve adjusted tests so they catch this problem.
>
>> Maybe we should add documentation to describe this behavior.
>
> I've added a paragraph about it to config.sgml, but I'm not sure about the comprehensiveness of the wording.
>
Thanks for updating the patch, no objections.
--
Regrads,
Japin Li
ChengDu WenWu Information Technology Co., Ltd.
From | Date | Subject | |
---|---|---|---|
Next Message | Andrey M. Borodin | 2023-12-18 09:40:31 | Re: Transaction timeout |
Previous Message | Peter Eisentraut | 2023-12-18 09:31:03 | Re: ALTER COLUMN ... SET EXPRESSION to alter stored generated column's expression |