Re: IGNITE-6430 (Done) CacheGroupsMetricsRebalanceTest.testRebalanceEstimateFinishTime test fails periodically

2018-05-15 Thread Dmitry Pavlov
Done. Alexander, Alexey, Dmitriy, thank you.

вт, 15 мая 2018 г. в 13:29, Dmitriy Govorukhin :

> Dmitriy,
>
> Looks good for me please merge.
>
> On Tue, May 15, 2018 at 11:20 AM, Dmitriy Govorukhin <
> dmitriy.govoruk...@gmail.com> wrote:
>
>> Dmitriy,
>>
>> Ok, I will do it soon.
>>
>> On Mon, May 14, 2018 at 8:41 PM, Dmitry Pavlov 
>> wrote:
>>
>>> Hi Dmitriy G,
>>>
>>> could you please pick up this review?
>>>
>>> Sincerely,
>>> Dmitriy Pavlov
>>>
>>> чт, 10 мая 2018 г. в 15:50, Александр Меньшиков :
>>>
 Hi,

 I have fixed problem with the test
 CacheGroupsMetricsRebalanceTest.testRebalanceEstimateFinishTime.

 Please review and merge if okay.

 P.S.
 Aleksey Plekhanov already approved it (see Jira).


 JIRA: https://issues.apache.org/jira/browse/IGNITE-6430
 PR: https://github.com/apache/ignite/pull/3914
 TC (with 120 passes):

 https://ci.ignite.apache.org/viewLog.html?buildId=1246058=IgniteTests24Java8_Cache3=testsInfo
 TC:

 https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_BuildApacheIgnite_IgniteTests24Java8=pull%2F3914%2Fhead=buildTypeStatusDiv

>>>
>>
>


Re: IGNITE-6430 (Done) CacheGroupsMetricsRebalanceTest.testRebalanceEstimateFinishTime test fails periodically

2018-05-15 Thread Dmitriy Govorukhin
Dmitriy,

Looks good for me please merge.

On Tue, May 15, 2018 at 11:20 AM, Dmitriy Govorukhin <
dmitriy.govoruk...@gmail.com> wrote:

> Dmitriy,
>
> Ok, I will do it soon.
>
> On Mon, May 14, 2018 at 8:41 PM, Dmitry Pavlov 
> wrote:
>
>> Hi Dmitriy G,
>>
>> could you please pick up this review?
>>
>> Sincerely,
>> Dmitriy Pavlov
>>
>> чт, 10 мая 2018 г. в 15:50, Александр Меньшиков :
>>
>>> Hi,
>>>
>>> I have fixed problem with the test
>>> CacheGroupsMetricsRebalanceTest.testRebalanceEstimateFinishTime.
>>>
>>> Please review and merge if okay.
>>>
>>> P.S.
>>> Aleksey Plekhanov already approved it (see Jira).
>>>
>>>
>>> JIRA: https://issues.apache.org/jira/browse/IGNITE-6430
>>> PR: https://github.com/apache/ignite/pull/3914
>>> TC (with 120 passes):
>>> https://ci.ignite.apache.org/viewLog.html?buildId=1246058
>>> ildTypeId=IgniteTests24Java8_Cache3=testsInfo
>>> TC:
>>> https://ci.ignite.apache.org/viewType.html?buildTypeId=Ignit
>>> eTests24Java8_BuildApacheIgnite_IgniteTests24Java8=
>>> pull%2F3914%2Fhead=buildTypeStatusDiv
>>>
>>
>


Re: IGNITE-6430 (Done) CacheGroupsMetricsRebalanceTest.testRebalanceEstimateFinishTime test fails periodically

2018-05-15 Thread Dmitriy Govorukhin
Dmitriy,

Ok, I will do it soon.

On Mon, May 14, 2018 at 8:41 PM, Dmitry Pavlov 
wrote:

> Hi Dmitriy G,
>
> could you please pick up this review?
>
> Sincerely,
> Dmitriy Pavlov
>
> чт, 10 мая 2018 г. в 15:50, Александр Меньшиков :
>
>> Hi,
>>
>> I have fixed problem with the test
>> CacheGroupsMetricsRebalanceTest.testRebalanceEstimateFinishTime.
>>
>> Please review and merge if okay.
>>
>> P.S.
>> Aleksey Plekhanov already approved it (see Jira).
>>
>>
>> JIRA: https://issues.apache.org/jira/browse/IGNITE-6430
>> PR: https://github.com/apache/ignite/pull/3914
>> TC (with 120 passes):
>> https://ci.ignite.apache.org/viewLog.html?buildId=1246058=
>> IgniteTests24Java8_Cache3=testsInfo
>> TC:
>> https://ci.ignite.apache.org/viewType.html?buildTypeId=
>> IgniteTests24Java8_BuildApacheIgnite_IgniteTests24Java8=pull%
>> 2F3914%2Fhead=buildTypeStatusDiv
>>
>


Re: IGNITE-6430 (Done) CacheGroupsMetricsRebalanceTest.testRebalanceEstimateFinishTime test fails periodically

2018-05-14 Thread Dmitry Pavlov
Hi Dmitriy G,

could you please pick up this review?

Sincerely,
Dmitriy Pavlov

чт, 10 мая 2018 г. в 15:50, Александр Меньшиков :

> Hi,
>
> I have fixed problem with the test
> CacheGroupsMetricsRebalanceTest.testRebalanceEstimateFinishTime.
>
> Please review and merge if okay.
>
> P.S.
> Aleksey Plekhanov already approved it (see Jira).
>
>
> JIRA: https://issues.apache.org/jira/browse/IGNITE-6430
> PR: https://github.com/apache/ignite/pull/3914
> TC (with 120 passes):
>
> https://ci.ignite.apache.org/viewLog.html?buildId=1246058=IgniteTests24Java8_Cache3=testsInfo
> TC:
>
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_BuildApacheIgnite_IgniteTests24Java8=pull%2F3914%2Fhead=buildTypeStatusDiv
>