Re: Transaction timeout

On Fri, 15 Dec 2023 at 17:51, Andrey M. Borodin <x4mmm@yandex-team.ru> wrote:
>> On 8 Dec 2023, at 15:29, Japin Li <japinli@hotmail.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.