Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.apache.org
Subject: Re: IGNITE-5123 Review
From: Evgeniy Ignatiev <yevgeniy.ignatyev@gmail.com>
To: dev@ignite.apache.org
References: <e6ca90b5-4159-4a88-7694-34ddb3fbfd49@gmail.com>
 <CAOKDqC-JoL7xN8QDJpWK0ZsV7przaPwxOMgT6gcXwOHZZfvH2Q@mail.gmail.com>
 <CALUCNEuZQURPiTOVQzJSjpr2Du+_4LJsdCeHPQybUscywsSswQ@mail.gmail.com>
 <64f82ec2-0635-d993-51fa-bada79a61bd5@gmail.com>
 <CALUCNEs2G0H8b_xt3jRXpYOnLWmfkrVZA00LuH_a_iSOrx+f_w@mail.gmail.com>
 <27c1f0b3-9c0e-9cf9-e3e6-ec0ee3ea78e9@gmail.com>
Message-ID: <4479e3e5-d450-8633-8c44-26f2523d5e33@gmail.com>
Date: Thu, 20 Jul 2017 12:48:08 +0400
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101
 Thunderbird/52.2.1
MIME-Version: 1.0
In-Reply-To: <27c1f0b3-9c0e-9cf9-e3e6-ec0ee3ea78e9@gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-US
archived-at: Thu, 20 Jul 2017 08:48:15 -0000

Rerun tests in "Run All" Ignite 2.0 group after merge with the latest 
master - 
http://ci.ignite.apache.org/viewLog.html?buildId=734705&tab=queuedBuildOverviewTab 
- Do you see any issues with them? (Unfortunately I have too little 
experience with Ignite tests to judge.)


On 7/19/2017 6:25 PM, Evgeniy Ignatiev wrote:
> Scheduled "Run All" in Ignite 2.0 Tests for pull/2285/head - 
> http://ci.ignite.apache.org/viewQueued.html?itemId=734208
>
>
> On 19.07.2017 17:55, Dmitry Pavlov wrote:
>> Hi Evgeniy,
>>
>> Thank you. Ignite Basic is one suite from approximately 80 suites that
>> covers Ignite by automated tests. Which is why I suggested to use RunAll
>> chain in ignite 2.0 group. Yes, several tests may fail, especially if 
>> it is
>> flaky tests or failure is related to the specific JIRA issue.
>>
>> About change itself: This change seems to be very impact. There is
>> possiblity that many of existing plugins relies on existing order of
>> initialization. This change may break plugin initialization in 
>> unexpected
>> manner.
>>
>> Could we
>> - fix javadoc according to existing order in code
>> - find out new solution?
>>
>> Sincerely,
>> Dmitriy Pavlov
>>
>>
>> ср, 19 июл. 2017 г. в 16:40, Evgeniy Ignatiev 
>> <yevgeniy.ignatyev@gmail.com>:
>>
>>> http://ci.ignite.apache.org/viewLog.html?buildId=720722&tab=buildResultsDiv&buildTypeId=IgniteTests_IgniteBasic 
>>>
>>> - this one - there seem to be no new failed platform tests, other 
>>> failed
>>> tests seem to fail in several other reviews too and are unrelated to my
>>> changes.
>>>
>>>
>>> On 19.07.2017 17:35, Dmitry Pavlov wrote:
>>>> Hi Evgeniy,
>>>>
>>>> I was not able to find Teamcity run for this change.
>>>> Could you please run http://ci.ignite.apache.org test for example on
>>> branch
>>>> pull/2285/head using 'Ignite 2.0 Tests' target 'Run All'.
>>>> Or could you please share link to previous run on this changes?
>>>>
>>>> Sincerely,
>>>> Dmitriy Pavlov
>>>>
>>>> ср, 19 июл. 2017 г. в 15:26, Anton Vinogradov <av@apache.org>:
>>>>
>>>>> Igniters,
>>>>>
>>>>> Could somebody review the fix today?
>>>>>
>>>>> On Wed, Jul 19, 2017 at 1:30 PM, Evgeniy Ignatiev <
>>>>> yevgeniy.ignatyev@gmail.com> wrote:
>>>>>
>>>>>> Hello, Igniters.
>>>>>>
>>>>>> Could anyone review my request - https://issues.apache.org/jira
>>>>>> /browse/IGNITE-5123? - My previous pings seems to got lost.
>>>>>>
>>>>>> Best regards,
>>>>>>
>>>>>> Yevgeniy
>>>>>>
>>>>>>
>>>
>