Renaming TcpDiscoverySpi.heartbeatsFrequency to TcpDiscoverySpi.metricsUpdateFrequency

[dmagda]

Yasha,

Thanks, I’ve replied you in the ticket.

In a nutshel, let’s try to merge current breaking changes to 2.0 and do the rest of compatible refactoring in further releases. Are you agree with this?

—
Denis

> On Apr 10, 2017, at 8:39 AM, Yakov Zhdanov <[hidden email]> wrote:
>
> Alex Belyak, please go ahead. Thank you for your help! There were some work
> I did not expect to happen.
>
>
> --Yakov
>
> 2017-04-10 18:13 GMT+03:00 Sasha Belyak <[hidden email]>:
>
>> Yakov, I can do replacement "heartbeat" -> "metricsUpdate". Can I get
>> https://issues.apache.org/jira/browse/IGNITE-4799 ?
>>
>> 2017-04-10 14:50 GMT+03:00 Yakov Zhdanov <[hidden email]>:
>>
>>> Guys, I have finished with initial changes. There were a little bit more
>>> work to do than I originally estimated.
>>>
>>> Denis, can you please reply in the ticket? I would like you to propose
>>> documentation changes. Can you help?
>>>
>>> There is also another question - we have a lot of "heartbeat" mentions in
>>> TCP disco - missed heartbeats, client heartbeats, heartbeat message.
>>> Should
>>> we go ahead and refactor this as well e.g. to maxMissedMetricUpdates?
>>>
>>> Also there are some changes to platforms and webconsole required. Alex
>>> Kuznetsov and Pavel Tupitsyn can you take a look at respective parts?
>>>
>>> --
>>> Yakov
>>>
>>
>>

[yzhdanov]

Messages renaming is never compatible =)

Denis, did you try to update javadocs?

--Yakov

2017-04-10 21:41 GMT+03:00 Denis Magda <[hidden email]>:

> Yasha,
>
> Thanks, I’ve replied you in the ticket.
>
> In a nutshel, let’s try to merge current breaking changes to 2.0 and do
> the rest of compatible refactoring in further releases. Are you agree with
> this?
>
> —
> Denis
>
> > On Apr 10, 2017, at 8:39 AM, Yakov Zhdanov <[hidden email]> wrote:
> >
> > Alex Belyak, please go ahead. Thank you for your help! There were some
> work
> > I did not expect to happen.
> >
> >
> > --Yakov
> >
> > 2017-04-10 18:13 GMT+03:00 Sasha Belyak <[hidden email]>:
> >
> >> Yakov, I can do replacement "heartbeat" -> "metricsUpdate". Can I get
> >> https://issues.apache.org/jira/browse/IGNITE-4799 ?
> >>
> >> 2017-04-10 14:50 GMT+03:00 Yakov Zhdanov <[hidden email]>:
> >>
> >>> Guys, I have finished with initial changes. There were a little bit
> more
> >>> work to do than I originally estimated.
> >>>
> >>> Denis, can you please reply in the ticket? I would like you to propose
> >>> documentation changes. Can you help?
> >>>
> >>> There is also another question - we have a lot of "heartbeat" mentions
> in
> >>> TCP disco - missed heartbeats, client heartbeats, heartbeat message.
> >>> Should
> >>> we go ahead and refactor this as well e.g. to maxMissedMetricUpdates?
> >>>
> >>> Also there are some changes to platforms and webconsole required. Alex
> >>> Kuznetsov and Pavel Tupitsyn can you take a look at respective parts?
> >>>
> >>> --
> >>> Yakov
> >>>
> >>
> >>
>
>

[dmagda]

Yasha,

I’ve updated the javadoc for IgniteConfiguration.metricsUpdateFrequency and merged the changes into your branch (see JIRA). The rest is fine for now. An advance documentation on how the heartbeats are implemented can be done in readme.io <http://readme.io/>.

—
Denis

> On Apr 11, 2017, at 1:07 AM, Yakov Zhdanov <[hidden email]> wrote:
>
> Messages renaming is never compatible =)
>
> Denis, did you try to update javadocs?
>
> --Yakov
>
> 2017-04-10 21:41 GMT+03:00 Denis Magda <[hidden email]>:
>
>> Yasha,
>>
>> Thanks, I’ve replied you in the ticket.
>>
>> In a nutshel, let’s try to merge current breaking changes to 2.0 and do
>> the rest of compatible refactoring in further releases. Are you agree with
>> this?
>>
>> —
>> Denis
>>
>>> On Apr 10, 2017, at 8:39 AM, Yakov Zhdanov <[hidden email]> wrote:
>>>
>>> Alex Belyak, please go ahead. Thank you for your help! There were some
>> work
>>> I did not expect to happen.
>>>
>>>
>>> --Yakov
>>>
>>> 2017-04-10 18:13 GMT+03:00 Sasha Belyak <[hidden email]>:
>>>
>>>> Yakov, I can do replacement "heartbeat" -> "metricsUpdate". Can I get
>>>> https://issues.apache.org/jira/browse/IGNITE-4799 ?
>>>>
>>>> 2017-04-10 14:50 GMT+03:00 Yakov Zhdanov <[hidden email]>:
>>>>
>>>>> Guys, I have finished with initial changes. There were a little bit
>> more
>>>>> work to do than I originally estimated.
>>>>>
>>>>> Denis, can you please reply in the ticket? I would like you to propose
>>>>> documentation changes. Can you help?
>>>>>
>>>>> There is also another question - we have a lot of "heartbeat" mentions
>> in
>>>>> TCP disco - missed heartbeats, client heartbeats, heartbeat message.
>>>>> Should
>>>>> we go ahead and refactor this as well e.g. to maxMissedMetricUpdates?
>>>>>
>>>>> Also there are some changes to platforms and webconsole required. Alex
>>>>> Kuznetsov and Pavel Tupitsyn can you take a look at respective parts?
>>>>>
>>>>> --
>>>>> Yakov
>>>>>
>>>>
>>>>
>>
>>

[dmagda]

Yakov, Sasha,

Are you ready to merge all the changes and close the ticket? As far as I understand, we plan to freeze 2.0 branch the next week.

—
Denis

> On Apr 11, 2017, at 8:59 AM, Denis Magda <[hidden email]> wrote:
>
> Yasha,
>
> I’ve updated the javadoc for IgniteConfiguration.metricsUpdateFrequency and merged the changes into your branch (see JIRA). The rest is fine for now. An advance documentation on how the heartbeats are implemented can be done in readme.io <http://readme.io/>.
>
> —
> Denis
>
>> On Apr 11, 2017, at 1:07 AM, Yakov Zhdanov <[hidden email]> wrote:
>>
>> Messages renaming is never compatible =)
>>
>> Denis, did you try to update javadocs?
>>
>> --Yakov
>>
>> 2017-04-10 21:41 GMT+03:00 Denis Magda <[hidden email]>:
>>
>>> Yasha,
>>>
>>> Thanks, I’ve replied you in the ticket.
>>>
>>> In a nutshel, let’s try to merge current breaking changes to 2.0 and do
>>> the rest of compatible refactoring in further releases. Are you agree with
>>> this?
>>>
>>> —
>>> Denis
>>>
>>>> On Apr 10, 2017, at 8:39 AM, Yakov Zhdanov <[hidden email]> wrote:
>>>>
>>>> Alex Belyak, please go ahead. Thank you for your help! There were some
>>> work
>>>> I did not expect to happen.
>>>>
>>>>
>>>> --Yakov
>>>>
>>>> 2017-04-10 18:13 GMT+03:00 Sasha Belyak <[hidden email]>:
>>>>
>>>>> Yakov, I can do replacement "heartbeat" -> "metricsUpdate". Can I get
>>>>> https://issues.apache.org/jira/browse/IGNITE-4799 ?
>>>>>
>>>>> 2017-04-10 14:50 GMT+03:00 Yakov Zhdanov <[hidden email]>:
>>>>>
>>>>>> Guys, I have finished with initial changes. There were a little bit
>>> more
>>>>>> work to do than I originally estimated.
>>>>>>
>>>>>> Denis, can you please reply in the ticket? I would like you to propose
>>>>>> documentation changes. Can you help?
>>>>>>
>>>>>> There is also another question - we have a lot of "heartbeat" mentions
>>> in
>>>>>> TCP disco - missed heartbeats, client heartbeats, heartbeat message.
>>>>>> Should
>>>>>> we go ahead and refactor this as well e.g. to maxMissedMetricUpdates?
>>>>>>
>>>>>> Also there are some changes to platforms and webconsole required. Alex
>>>>>> Kuznetsov and Pavel Tupitsyn can you take a look at respective parts?
>>>>>>
>>>>>> --
>>>>>> Yakov
>>>>>>
>>>>>
>>>>>
>>>
>>>
>

[yzhdanov]

Denis, there were minor test failures on tc. Alex will fix them on Mon and
I expect merge to happen right after.

--
Yakov Zhdanov

On Apr 22, 2017 2:22 AM, "Denis Magda" <[hidden email]> wrote:

> Yakov, Sasha,
>
> Are you ready to merge all the changes and close the ticket? As far as I
> understand, we plan to freeze 2.0 branch the next week.
>
> —
> Denis
>
> > On Apr 11, 2017, at 8:59 AM, Denis Magda <[hidden email]> wrote:
> >
> > Yasha,
> >
> > I’ve updated the javadoc for IgniteConfiguration.metricsUpdateFrequency
> and merged the changes into your branch (see JIRA). The rest is fine for
> now. An advance documentation on how the heartbeats are implemented can be
> done in readme.io <http://readme.io/>.
> >
> > —
> > Denis
> >
> >> On Apr 11, 2017, at 1:07 AM, Yakov Zhdanov <[hidden email]> wrote:
> >>
> >> Messages renaming is never compatible =)
> >>
> >> Denis, did you try to update javadocs?
> >>
> >> --Yakov
> >>
> >> 2017-04-10 21:41 GMT+03:00 Denis Magda <[hidden email]>:
> >>
> >>> Yasha,
> >>>
> >>> Thanks, I’ve replied you in the ticket.
> >>>
> >>> In a nutshel, let’s try to merge current breaking changes to 2.0 and do
> >>> the rest of compatible refactoring in further releases. Are you agree
> with
> >>> this?
> >>>
> >>> —
> >>> Denis
> >>>
> >>>> On Apr 10, 2017, at 8:39 AM, Yakov Zhdanov <[hidden email]>
> wrote:
> >>>>
> >>>> Alex Belyak, please go ahead. Thank you for your help! There were some
> >>> work
> >>>> I did not expect to happen.
> >>>>
> >>>>
> >>>> --Yakov
> >>>>
> >>>> 2017-04-10 18:13 GMT+03:00 Sasha Belyak <[hidden email]>:
> >>>>
> >>>>> Yakov, I can do replacement "heartbeat" -> "metricsUpdate". Can I get
> >>>>> https://issues.apache.org/jira/browse/IGNITE-4799 ?
> >>>>>
> >>>>> 2017-04-10 14:50 GMT+03:00 Yakov Zhdanov <[hidden email]>:
> >>>>>
> >>>>>> Guys, I have finished with initial changes. There were a little bit
> >>> more
> >>>>>> work to do than I originally estimated.
> >>>>>>
> >>>>>> Denis, can you please reply in the ticket? I would like you to
> propose
> >>>>>> documentation changes. Can you help?
> >>>>>>
> >>>>>> There is also another question - we have a lot of "heartbeat"
> mentions
> >>> in
> >>>>>> TCP disco - missed heartbeats, client heartbeats, heartbeat message.
> >>>>>> Should
> >>>>>> we go ahead and refactor this as well e.g. to
> maxMissedMetricUpdates?
> >>>>>>
> >>>>>> Also there are some changes to platforms and webconsole required.
> Alex
> >>>>>> Kuznetsov and Pavel Tupitsyn can you take a look at respective
> parts?
> >>>>>>
> >>>>>> --
> >>>>>> Yakov
> >>>>>>
> >>>>>
> >>>>>
> >>>
> >>>
> >
>
>