| 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-16 00:58:53 |
| Message-ID: | ME3P282MB3166E33F193D2E7B0CDB633FB692A@ME3P282MB3166.AUSP282.PROD.OUTLOOK.COM |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
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?
Maybe we should add documentation to describe this behavior.
> Consider changing status of the commitfest entry if you think it’s ready for committer.
>
--
Regrads,
Japin Li
ChengDu WenWu Information Technology Co., Ltd.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2023-12-16 01:23:53 | Re: encoding affects ICU regex character classification |
| Previous Message | Jeremy Schneider | 2023-12-16 00:48:23 | Re: encoding affects ICU regex character classification |