You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Zhenya Stanilovsky <ar...@mail.ru.INVALID> on 2020/05/22 15:20:02 UTC

Re[2]: Proposal: set default transaction timeout to 5 minutes

Ivan, does global timeout change through jmx in scope of this ticket ? If so, can you add it ? Opposite we need additional ticket, i hope ? We still have no somehow store for jmx changed params, every one need to remember that cluster restart will reset this setting to default, in this case system param need to be appended.


 
>https://issues.apache.org/jira/browse/IGNITE-13064 is raised with label
>"newbie".
>
>On Tue, May 19, 2020 at 4:10 PM Ivan Rakov < ivan.glukos@gmail.com > wrote:
> 
>> Support this idea in general but why 5 minutes and not less?
>>
>> This value looks to me greater than any value that can possibly affect
>> existing deployments (existing long transactions may suddenly start to
>> rollback), but less than reaction time of users that are only starting to
>> get along with Ignite and suddenly experience TX deadlock.
>>
>> --
>> Best Regards,
>> Ivan Rakov
>>
>> On Tue, May 19, 2020 at 10:31 AM Anton Vinogradov < av@apache.org > wrote:
>>
>>> +1
>>>
>>> On Mon, May 18, 2020 at 9:45 PM Sergey Antonov < antonovsergey93@gmail.com
>>> >
>>> wrote:
>>>
>>> > +1
>>> >
>>> > пн, 18 мая 2020 г. в 21:26, Andrey Mashenkov <
>>>  andrey.mashenkov@gmail.com >:
>>> >
>>> > > +1
>>> > >
>>> > > On Mon, May 18, 2020 at 9:19 PM Ivan Rakov < ivan.glukos@gmail.com >
>>> > wrote:
>>> > >
>>> > > > Hi Igniters,
>>> > > >
>>> > > > I have a very simple proposal. Let's set default TX timeout to 5
>>> > minutes
>>> > > > (right now it's 0 = no timeout).
>>> > > > Pros:
>>> > > > 1. Deadlock detection procedure is triggered on timeout. In case
>>> user
>>> > > will
>>> > > > get into key-level deadlock, he'll be able to discover root cause
>>> from
>>> > > the
>>> > > > logs (even though load will hang for a while) and skip step with
>>> > googling
>>> > > > and debugging.
>>> > > > 2. Almost every system with transactions has timeout enabled by
>>> > default.
>>> > > >
>>> > > > WDYT?
>>> > > >
>>> > > > --
>>> > > > Best Regards,
>>> > > > Ivan Rakov
>>> > > >
>>> > >
>>> > >
>>> > > --
>>> > > Best regards,
>>> > > Andrey V. Mashenkov
>>> > >
>>> >
>>> >
>>> > --
>>> > BR, Sergey Antonov
>>> >
>>>
>> 
 
 
 
 

Re[4]: Proposal: set default transaction timeout to 5 minutes

Posted by Zhenya Stanilovsky <ar...@mail.ru.INVALID>.
Of course, i just thinking about huge persistent installation and guys who not carefully reads Release Notes )
In case of long tx timeouts by design, they can easily fix default timeout with just one jmx call. 
 
>Zhenya,
>
>Can you please elaborate?
>Why we need to change default TX timeout via JMX? It looks feasible and
>perhaps may work as a hotfix for live deployments experiencing issues with
>long transactions, but it's definitely a separate issue.
>
>On Fri, May 22, 2020 at 6:20 PM Zhenya Stanilovsky
>< arzamas123@mail.ru.invalid > wrote:
> 
>>
>> Ivan, does global timeout change through jmx in scope of this ticket ? If
>> so, can you add it ? Opposite we need additional ticket, i hope ? We
>> still have no somehow store for jmx changed params, every one need to
>> remember that cluster restart will reset this setting to default, in this
>> case system param need to be appended.
>>
>>
>>
>> > https://issues.apache.org/jira/browse/IGNITE-13064 is raised with label
>> >"newbie".
>> >
>> >On Tue, May 19, 2020 at 4:10 PM Ivan Rakov <  ivan.glukos@gmail.com >
>> wrote:
>> >
>> >> Support this idea in general but why 5 minutes and not less?
>> >>
>> >> This value looks to me greater than any value that can possibly affect
>> >> existing deployments (existing long transactions may suddenly start to
>> >> rollback), but less than reaction time of users that are only starting
>> to
>> >> get along with Ignite and suddenly experience TX deadlock.
>> >>
>> >> --
>> >> Best Regards,
>> >> Ivan Rakov
>> >>
>> >> On Tue, May 19, 2020 at 10:31 AM Anton Vinogradov <  av@apache.org >
>> wrote:
>> >>
>> >>> +1
>> >>>
>> >>> On Mon, May 18, 2020 at 9:45 PM Sergey Antonov <
>>  antonovsergey93@gmail.com
>> >>> >
>> >>> wrote:
>> >>>
>> >>> > +1
>> >>> >
>> >>> > пн, 18 мая 2020 г. в 21:26, Andrey Mashenkov <
>> >>>  andrey.mashenkov@gmail.com >:
>> >>> >
>> >>> > > +1
>> >>> > >
>> >>> > > On Mon, May 18, 2020 at 9:19 PM Ivan Rakov <  ivan.glukos@gmail.com
>> >
>> >>> > wrote:
>> >>> > >
>> >>> > > > Hi Igniters,
>> >>> > > >
>> >>> > > > I have a very simple proposal. Let's set default TX timeout to 5
>> >>> > minutes
>> >>> > > > (right now it's 0 = no timeout).
>> >>> > > > Pros:
>> >>> > > > 1. Deadlock detection procedure is triggered on timeout. In case
>> >>> user
>> >>> > > will
>> >>> > > > get into key-level deadlock, he'll be able to discover root cause
>> >>> from
>> >>> > > the
>> >>> > > > logs (even though load will hang for a while) and skip step with
>> >>> > googling
>> >>> > > > and debugging.
>> >>> > > > 2. Almost every system with transactions has timeout enabled by
>> >>> > default.
>> >>> > > >
>> >>> > > > WDYT?
>> >>> > > >
>> >>> > > > --
>> >>> > > > Best Regards,
>> >>> > > > Ivan Rakov
>> >>> > > >
>> >>> > >
>> >>> > >
>> >>> > > --
>> >>> > > Best regards,
>> >>> > > Andrey V. Mashenkov
>> >>> > >
>> >>> >
>> >>> >
>> >>> > --
>> >>> > BR, Sergey Antonov
>> >>> >
>> >>>
>> >>
>>
>>
>>
>> 
 
 
 
 

Re: Re[2]: Proposal: set default transaction timeout to 5 minutes

Posted by Ivan Rakov <iv...@gmail.com>.
Zhenya,

Can you please elaborate?
Why we need to change default TX timeout via JMX? It looks feasible and
perhaps may work as a hotfix for live deployments experiencing issues with
long transactions, but it's definitely a separate issue.

On Fri, May 22, 2020 at 6:20 PM Zhenya Stanilovsky
<ar...@mail.ru.invalid> wrote:

>
> Ivan, does global timeout change through jmx in scope of this ticket ? If
> so, can you add it ? Opposite we need additional ticket, i hope ? We
> still have no somehow store for jmx changed params, every one need to
> remember that cluster restart will reset this setting to default, in this
> case system param need to be appended.
>
>
>
> >https://issues.apache.org/jira/browse/IGNITE-13064 is raised with label
> >"newbie".
> >
> >On Tue, May 19, 2020 at 4:10 PM Ivan Rakov < ivan.glukos@gmail.com >
> wrote:
> >
> >> Support this idea in general but why 5 minutes and not less?
> >>
> >> This value looks to me greater than any value that can possibly affect
> >> existing deployments (existing long transactions may suddenly start to
> >> rollback), but less than reaction time of users that are only starting
> to
> >> get along with Ignite and suddenly experience TX deadlock.
> >>
> >> --
> >> Best Regards,
> >> Ivan Rakov
> >>
> >> On Tue, May 19, 2020 at 10:31 AM Anton Vinogradov < av@apache.org >
> wrote:
> >>
> >>> +1
> >>>
> >>> On Mon, May 18, 2020 at 9:45 PM Sergey Antonov <
> antonovsergey93@gmail.com
> >>> >
> >>> wrote:
> >>>
> >>> > +1
> >>> >
> >>> > пн, 18 мая 2020 г. в 21:26, Andrey Mashenkov <
> >>>  andrey.mashenkov@gmail.com >:
> >>> >
> >>> > > +1
> >>> > >
> >>> > > On Mon, May 18, 2020 at 9:19 PM Ivan Rakov < ivan.glukos@gmail.com
> >
> >>> > wrote:
> >>> > >
> >>> > > > Hi Igniters,
> >>> > > >
> >>> > > > I have a very simple proposal. Let's set default TX timeout to 5
> >>> > minutes
> >>> > > > (right now it's 0 = no timeout).
> >>> > > > Pros:
> >>> > > > 1. Deadlock detection procedure is triggered on timeout. In case
> >>> user
> >>> > > will
> >>> > > > get into key-level deadlock, he'll be able to discover root cause
> >>> from
> >>> > > the
> >>> > > > logs (even though load will hang for a while) and skip step with
> >>> > googling
> >>> > > > and debugging.
> >>> > > > 2. Almost every system with transactions has timeout enabled by
> >>> > default.
> >>> > > >
> >>> > > > WDYT?
> >>> > > >
> >>> > > > --
> >>> > > > Best Regards,
> >>> > > > Ivan Rakov
> >>> > > >
> >>> > >
> >>> > >
> >>> > > --
> >>> > > Best regards,
> >>> > > Andrey V. Mashenkov
> >>> > >
> >>> >
> >>> >
> >>> > --
> >>> > BR, Sergey Antonov
> >>> >
> >>>
> >>
>
>
>
>