Hi Junwang Zhao
#should we invalidate lock_timeout? Or maybe just document this.
I think you mean when lock_time is greater than trasaction-time invalidate
lock_timeout or needs to be logged ?
Best whish
________________________________
发件人: Junwang Zhao <[email protected]>
发送时间: 2023年12月20日 9:48
收件人: Andrey M. Borodin <[email protected]>
抄送: Japin Li <[email protected]>; 邱宇航 <[email protected]>; Fujii Masao
<[email protected]>; Andrey Borodin <[email protected]>; Andres
Freund <[email protected]>; Michael Paquier <[email protected]>;
Nikolay Samokhvalov <[email protected]>; pgsql-hackers
<[email protected]>; [email protected]
<[email protected]>
主题: Re: Transaction timeout
On Tue, Dec 19, 2023 at 10:51 PM Junwang Zhao <[email protected]> wrote:
>
> On Tue, Dec 19, 2023 at 6:27 PM Andrey M. Borodin <[email protected]>
> wrote:
> >
> >
> >
> > > On 19 Dec 2023, at 13:26, Andrey M. Borodin <[email protected]> wrote:
> > >
> > > I don’t have Windows machine, so I hope CF bot will pick this.
> >
> > I used Github CI to produce version of tests that seems to be is stable on
> > Windows.
> > Sorry for the noise.
> >
> >
> > Best regards, Andrey Borodin.
>
> + <para>
> + If <varname>transaction_timeout</varname> is shorter than
> + <varname>idle_in_transaction_session_timeout</varname> or
> <varname>statement_timeout</varname>
> + <varname>transaction_timeout</varname> will invalidate longer
> timeout.
> + </para>
>
> When transaction_timeout is *equal* to idle_in_transaction_session_timeout
> or statement_timeout, idle_in_transaction_session_timeout and
> statement_timeout
> will also be invalidated, the logic in the code seems right, though
> this document
> is a little bit inaccurate.
>
<para>
Unlike <varname>statement_timeout</varname>, this timeout can only occur
while waiting for locks. Note that if
<varname>statement_timeout</varname>
is nonzero, it is rather pointless to set
<varname>lock_timeout</varname> to
the same or larger value, since the statement timeout would always
trigger first. If <varname>log_min_error_statement</varname> is set to
<literal>ERROR</literal> or lower, the statement that timed out will be
logged.
</para>
There is a note about statement_timeout and lock_timeout, set both
and lock_timeout >= statement_timeout is pointless, but this logic seems not
implemented in the code. I am wondering if lock_timeout >= transaction_timeout,
should we invalidate lock_timeout? Or maybe just document this.
> --
> Regards
> Junwang Zhao
--
Regards
Junwang Zhao
