You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Ivan Rakov <iv...@gmail.com> on 2020/05/18 18:19:26 UTC

Proposal: set default transaction timeout to 5 minutes

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

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
> >>> >
> >>>
> >>
>
>
>
>

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

Posted by Zhenya Stanilovsky <ar...@mail.ru.INVALID>.
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: Proposal: set default transaction timeout to 5 minutes

Posted by Ivan Rakov <iv...@gmail.com>.
https://issues.apache.org/jira/browse/IGNITE-13064 is raised with label
"newbie".

On Tue, May 19, 2020 at 4:10 PM Ivan Rakov <iv...@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 <iv...@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: Proposal: set default transaction timeout to 5 minutes

Posted by Ivan Rakov <iv...@gmail.com>.
>
> 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 <an...@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 <iv...@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: Proposal: set default transaction timeout to 5 minutes

Posted by Anton Vinogradov <av...@apache.org>.
+1

On Mon, May 18, 2020 at 9:45 PM Sergey Antonov <an...@gmail.com>
wrote:

> +1
>
> пн, 18 мая 2020 г. в 21:26, Andrey Mashenkov <an...@gmail.com>:
>
> > +1
> >
> > On Mon, May 18, 2020 at 9:19 PM Ivan Rakov <iv...@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: Proposal: set default transaction timeout to 5 minutes

Posted by Sergey Antonov <an...@gmail.com>.
+1

пн, 18 мая 2020 г. в 21:26, Andrey Mashenkov <an...@gmail.com>:

> +1
>
> On Mon, May 18, 2020 at 9:19 PM Ivan Rakov <iv...@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: Proposal: set default transaction timeout to 5 minutes

Posted by Andrey Mashenkov <an...@gmail.com>.
+1

On Mon, May 18, 2020 at 9:19 PM Ivan Rakov <iv...@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

Re: Proposal: set default transaction timeout to 5 minutes

Posted by Zhenya Stanilovsky <ar...@mail.ru.INVALID>.
+1 here.

> 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

Re: Proposal: set default transaction timeout to 5 minutes

Posted by Denis Magda <dm...@apache.org>.
Hi Ivan,

Support this idea in general but why 5 minutes and not less?

-
Denis


On Mon, May 18, 2020 at 11:19 AM Ivan Rakov <iv...@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
>