Hi, Petr! Profile "check-test-suites" handles all tests in another way, it just verifies that all tests are suited. No tests run then. As I understand the [BUILD] job goal is preparing a .zip archive to distribute it for other jobs. I think it is a valid place to put sanity checks. If a check fails then no archive is prepared. WDYT?
Also I see that there is a flag -Dmaven.javadoc.skip=true. I'd propose to change it to the profile "skip-docs", that was introduced in ticket [1] IGNITE-13623. As the setting "maven.javadoc.skip" does not affect scaladocs. [1] https://issues.apache.org/jira/browse/IGNITE-13623 On Thu, Feb 25, 2021 at 7:34 AM Petr Ivanov <piva...@gridgain.com> wrote: > Won't the absence of -DskipTests flag trigger ALL the tests for all > modules? > This flag was added intentionally. > > Instead, I'd put Non-Suited tests into some kind of sanity check, group > all sanity checks in single Run All, and make tests depend on it's > successful pass. > > > Regards, > *Petr Ivanov* > Head of IT > IT & Development Solutions | > *GRIDGAIN SYSTEMS*+7 (911) 945-00-59 > > On 24 Feb 2021, at 19:58, Max Timonin <timonin.ma...@gmail.com> wrote: > > Hi, all! > > What do you think if we add the check in the TC [Build] job. Currently > [Build] runs also check for licences, checkstyle [1]: > > mvn *install* -Pall-java,all-scala,scala,*licenses*,lgpl,examples, > *checkstyle* -DskipTests -Dmaven.javadoc.skip=true %MAVEN_MODULES_STRING%. > > So let's add the check too to block other jobs. As if there missed tests > then TC run may be invalid - missed tests may be broken and then the MTCGA > visa too. To made this we should change command line parameters: > 1. Add profile check-test-suites; > 2. Remove -Dskiptests flag. > > -Pall-java,all-scala,scala,licenses,lgpl,examples,checkstyle, > *check-test-suites *-DskipTests -Dmaven.javadoc.skip=true > %MAVEN_MODULES_STRING% > > WDYT? > > [1] > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_BuildApacheIgnite&tab=buildTypeSettings&branch_IgniteTests24Java8=%3Cdefault > > On Tue, Feb 9, 2021 at 4:48 PM Ilya Kasnacheev <ilya.kasnach...@gmail.com> > wrote: > >> Hello again! >> >> Of course it's 20 minutes, not 20 seconds. >> >> Regards, >> -- >> Ilya Kasnacheev >> >> >> вт, 9 февр. 2021 г. в 16:45, Ilya Kasnacheev <ilya.kasnach...@gmail.com>: >> >> > Hello! >> > >> > Java part kicks in if the target not found in pom.xml. Ideally we should >> > skip this build if target check-test-suites is not in pom.xml >> > >> > I have changed its timeout to 20 second which should terminate its >> > progression on older builds. Maybe that would be sufficient for now. >> > >> > Regards, >> > -- >> > Ilya Kasnacheev >> > >> > >> > вт, 9 февр. 2021 г. в 14:09, Petr Ivanov <piva...@gridgain.com>: >> > >> >> As much as I understood — we execute internal class as plugin, where >> all >> >> the magic is done. >> >> Seems pretty solid in Maven part. Java part, unfortunately, cannot >> check. >> >> >> >> >> >> >> >> Regards, >> >> *Petr Ivanov* >> >> Head of IT >> >> IT & Development Solutions | >> >> *GRIDGAIN SYSTEMS*+7 (911) 945-00-59 >> >> >> >> On 9 Feb 2021, at 12:32, Ilya Kasnacheev <ilya.kasnach...@gmail.com> >> >> wrote: >> >> >> >> Hello Peter, >> >> >> >> Thanks for chiming in. The code is under >> >> https://github.com/apache/ignite/pull/8367 >> >> >> >> Regards, >> >> -- >> >> Ilya Kasnacheev >> >> >> >> >> >> вт, 9 февр. 2021 г. в 12:20, Petr Ivanov <piva...@gridgain.com>: >> >> >> >>> Hi, Ilya. >> >>> >> >>> >> >>> I've added Inspections to Run All. >> >>> Checkstyle is currently integrated to Build and can be deleted. >> >>> >> >>> >> >>> Where can I find the code for check-test-suites profile? >> >>> >> >>> >> >>> Regards, >> >>> *Petr Ivanov* >> >>> Head of IT >> >>> IT & Development Solutions | >> >>> *GRIDGAIN SYSTEMS*+7 (911) 945-00-59 >> >>> >> >>> On 9 Feb 2021, at 12:16, Ilya Kasnacheev <ilya.kasnach...@gmail.com> >> >>> wrote: >> >>> >> >>> Hello! >> >>> >> >>> I have found one issue where it would cause tests to be run if the >> >>> change is not present in the target branch. >> >>> >> >>> This includes e.g. 2.10 nightlies. >> >>> >> >>> What can we do to avoid that? Is specifying -DskipTests sufficient? >> Any >> >>> chance that it will break the missed tests check? >> >>> >> >>> Regards, >> >>> -- >> >>> Ilya Kasnacheev >> >>> >> >>> >> >>> пн, 8 февр. 2021 г. в 14:13, Ilya Kasnacheev < >> ilya.kasnach...@gmail.com >> >>> >: >> >>> >> >>>> Hello! >> >>>> >> >>>> I have created a TC suite: >> >>>> >> >>>> >> https://ci.ignite.apache.org/buildConfiguration/IgniteTests24Java8_MissingTests?mode=builds >> >>>> >> >>>> + Peter Ivanov >> >>>> >> >>>> Can you please check if everything is alright? >> >>>> >> >>>> BTW, it seems that Inspections [Core] is only in Run All Basic (but >> not >> >>>> in Run All), and Check Code Style is not triggered by either one. Is >> it >> >>>> correct? >> >>>> >> >>>> Regards, >> >>>> -- >> >>>> Ilya Kasnacheev >> >>>> >> >>>> >> >>>> пн, 8 февр. 2021 г. в 10:22, Max Timonin <timonin.ma...@gmail.com>: >> >>>> >> >>>>> Hi! >> >>>>> >> >>>>> Yes, now it's a part of the Travis check, and there is already a >> first >> >>>>> successful build [1]. But I think it's also required to run the >> check >> >>>>> on TC >> >>>>> too, along with jobs for checking licenses, code style, and core >> >>>>> inspections. >> >>>>> >> >>>>> >> >>>>> [1] https://travis-ci.com/github/apache/ignite/builds/216363067 >> >>>>> >> >>>>> On Fri, Feb 5, 2021 at 7:13 PM Ilya Kasnacheev < >> >>>>> ilya.kasnach...@gmail.com> >> >>>>> wrote: >> >>>>> >> >>>>> > Hello! >> >>>>> > >> >>>>> > I have merged it to master! >> >>>>> > >> >>>>> > I wonder what happens next. It will run as a part of travis check? >> >>>>> Do we >> >>>>> > also need to add it as a TC suite? >> >>>>> > >> >>>>> > Regards, >> >>>>> > -- >> >>>>> > Ilya Kasnacheev >> >>>>> > >> >>>>> > >> >>>>> > ср, 3 февр. 2021 г. в 18:50, Ilya Kasnacheev < >> >>>>> ilya.kasnach...@gmail.com>: >> >>>>> > >> >>>>> > > Hello! >> >>>>> > > >> >>>>> > > Code mostly looks good, I have added a minor request. I will >> check >> >>>>> how it >> >>>>> > > works and then we may commit. >> >>>>> > > >> >>>>> > > + zaleslaw....@gmail.com >> >>>>> > > >> >>>>> > > Can you please check whether the new ML suites make sense? >> >>>>> > > math/distances/DistancesTestSuite.java >> >>>>> > > naivebayes/NaiveBayesTestSuite.java >> >>>>> > > >> >>>>> > > Would we need to add them to some TC runs? >> >>>>> > > >> >>>>> > > Regards, >> >>>>> > > -- >> >>>>> > > Ilya Kasnacheev >> >>>>> > > >> >>>>> > > >> >>>>> > > пн, 25 янв. 2021 г. в 22:07, Max Timonin < >> timonin.ma...@gmail.com >> >>>>> >: >> >>>>> > > >> >>>>> > >> Hi, Ilya! >> >>>>> > >> >> >>>>> > >> I made a fix to the check. Now it aggregates info about tests >> and >> >>>>> suites >> >>>>> > >> from all modules and then validates it. Could you please review >> >>>>> the PR >> >>>>> > >> [1]? >> >>>>> > >> >> >>>>> > >> I tried to move some tests between modules, but unfortunately >> it >> >>>>> still >> >>>>> > >> looks like spaghetti. So I reverted all changes to testsuites >> >>>>> (new and >> >>>>> > >> splitted suites) and reworked the check. >> >>>>> > >> >> >>>>> > >> [1] https://github.com/apache/ignite/pull/8367 >> >>>>> > >> >> >>>>> > >> On Mon, Dec 28, 2020 at 2:03 PM Ilya Kasnacheev < >> >>>>> > >> ilya.kasnach...@gmail.com> >> >>>>> > >> wrote: >> >>>>> > >> >> >>>>> > >> > Hello! >> >>>>> > >> > >> >>>>> > >> > You could try to move these tests as .java files between >> >>>>> modules in a >> >>>>> > >> > separate commit. I think I could review it. >> >>>>> > >> > >> >>>>> > >> > Regards, >> >>>>> > >> > -- >> >>>>> > >> > Ilya Kasnacheev >> >>>>> > >> > >> >>>>> > >> > >> >>>>> > >> > пт, 25 дек. 2020 г. в 17:19, Max Timonin < >> >>>>> timonin.ma...@gmail.com>: >> >>>>> > >> > >> >>>>> > >> > > Hi! >> >>>>> > >> > > >> >>>>> > >> > > Ilya thanks for the reply! I agree that it's a valid case >> >>>>> when a >> >>>>> > test >> >>>>> > >> is >> >>>>> > >> > > part of multiple suites in different modules. But it is >> >>>>> definitely a >> >>>>> > >> bug >> >>>>> > >> > > that the test is written in a module where it can't be run >> at >> >>>>> all >> >>>>> > and >> >>>>> > >> > aimed >> >>>>> > >> > > to run within different modules (core tests in core that >> >>>>> require >> >>>>> > h2). >> >>>>> > >> I >> >>>>> > >> > > propose to fix this issue. >> >>>>> > >> > > >> >>>>> > >> > > I'm going to check all such tests and move them to the >> right >> >>>>> module. >> >>>>> > >> As I >> >>>>> > >> > > can see there are about 100 such test classes, but I hope >> >>>>> that most >> >>>>> > of >> >>>>> > >> > them >> >>>>> > >> > > follow only a few patterns. >> >>>>> > >> > > >> >>>>> > >> > > On Fri, Dec 25, 2020 at 2:58 PM Ivan Daschinsky < >> >>>>> > ivanda...@gmail.com> >> >>>>> > >> > > wrote: >> >>>>> > >> > > >> >>>>> > >> > > > Hi! >> >>>>> > >> > > > >> I'm not sure that we should assume every test is only >> >>>>> run from >> >>>>> > >> one >> >>>>> > >> > > test >> >>>>> > >> > > > suite. One test may be run from different test suites >> >>>>> located in >> >>>>> > >> > > different >> >>>>> > >> > > > modules. >> >>>>> > >> > > > Agree. For example, if we introduce this limitation, zk >> >>>>> suites >> >>>>> > will >> >>>>> > >> be >> >>>>> > >> > > > broken. >> >>>>> > >> > > > >> >>>>> > >> > > > >> >>>>> > >> > > > пт, 25 дек. 2020 г. в 14:48, Ilya Kasnacheev < >> >>>>> > >> > ilya.kasnach...@gmail.com >> >>>>> > >> > > >: >> >>>>> > >> > > > >> >>>>> > >> > > > > Hello! >> >>>>> > >> > > > > >> >>>>> > >> > > > > Sorry for the long wait. >> >>>>> > >> > > > > >> >>>>> > >> > > > > I'm not sure that we should assume every test is only >> run >> >>>>> from >> >>>>> > one >> >>>>> > >> > test >> >>>>> > >> > > > > suite. One test may be run from different test suites >> >>>>> located in >> >>>>> > >> > > > different >> >>>>> > >> > > > > modules. >> >>>>> > >> > > > > >> >>>>> > >> > > > > I wonder if we can drop this requirement, check all the >> >>>>> modules >> >>>>> > >> > > > > transitively for used/unused tests. >> >>>>> > >> > > > > >> >>>>> > >> > > > > Regards, >> >>>>> > >> > > > > -- >> >>>>> > >> > > > > Ilya Kasnacheev >> >>>>> > >> > > > > >> >>>>> > >> > > > > >> >>>>> > >> > > > > ср, 2 дек. 2020 г. в 18:23, Max Timonin < >> >>>>> > timonin.ma...@gmail.com >> >>>>> > >> >: >> >>>>> > >> > > > > >> >>>>> > >> > > > > > Hi, >> >>>>> > >> > > > > > >> >>>>> > >> > > > > > I don't think so. It looks like a bug that tests fail >> >>>>> if one >> >>>>> > >> runs >> >>>>> > >> > > them >> >>>>> > >> > > > > > within their module (actually, what is the goal of >> this >> >>>>> > test?). >> >>>>> > >> The >> >>>>> > >> > > > check >> >>>>> > >> > > > > > showed us this problem, there is no need to fix the >> >>>>> check. >> >>>>> > >> > > > > > >> >>>>> > >> > > > > > Currently I see two ways: >> >>>>> > >> > > > > > 1. Find the right module for every misplaced test. >> >>>>> There are >> >>>>> > 104 >> >>>>> > >> > > tests, >> >>>>> > >> > > > > > maybe just move them all to the target module? If >> >>>>> TeamCity >> >>>>> > runs >> >>>>> > >> > them >> >>>>> > >> > > > > within >> >>>>> > >> > > > > > the indexing module only is there a reason to have a >> >>>>> test in >> >>>>> > the >> >>>>> > >> > core >> >>>>> > >> > > > > > module at all? >> >>>>> > >> > > > > > 2. Back to my previous proposal - create fake suites >> >>>>> within a >> >>>>> > >> > module, >> >>>>> > >> > > > > then >> >>>>> > >> > > > > > replace test classes in a target suite with the >> single >> >>>>> class >> >>>>> > of >> >>>>> > >> the >> >>>>> > >> > > > fake >> >>>>> > >> > > > > > suite. >> >>>>> > >> > > > > > >> >>>>> > >> > > > > > >> >>>>> > >> > > > > > >> >>>>> > >> > > > > > On Wed, Dec 2, 2020 at 5:38 PM Ilya Kasnacheev < >> >>>>> > >> > > > > ilya.kasnach...@gmail.com> >> >>>>> > >> > > > > > wrote: >> >>>>> > >> > > > > > >> >>>>> > >> > > > > > > Hello! >> >>>>> > >> > > > > > > >> >>>>> > >> > > > > > > I think this means that we should abandon the plan >> of >> >>>>> moving >> >>>>> > >> > tests >> >>>>> > >> > > > > > between >> >>>>> > >> > > > > > > suites, and that your tool has to understand the >> >>>>> dependency >> >>>>> > >> graph >> >>>>> > >> > > > > > > between modules' tests when assessing what's >> included >> >>>>> and >> >>>>> > >> what's >> >>>>> > >> > > not. >> >>>>> > >> > > > > > > >> >>>>> > >> > > > > > > Regards, >> >>>>> > >> > > > > > > -- >> >>>>> > >> > > > > > > Ilya Kasnacheev >> >>>>> > >> > > > > > > >> >>>>> > >> > > > > > > >> >>>>> > >> > > > > > > ср, 2 дек. 2020 г. в 15:56, Max Timonin < >> >>>>> > >> timonin.ma...@gmail.com >> >>>>> > >> > >: >> >>>>> > >> > > > > > > >> >>>>> > >> > > > > > > > Hi, Ilya! >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > I've checked testsuites. There is an issue. For >> >>>>> example >> >>>>> > >> > > > > > > > *IgniteBinaryCacheQueryTestSuite* suite is now >> in 2 >> >>>>> > modules: >> >>>>> > >> > > > > > ignite-core, >> >>>>> > >> > > > > > > > ignite-indexing. On TeamCity it runs by "Query 1" >> >>>>> suite. >> >>>>> > >> > > Simplified >> >>>>> > >> > > > > > maven >> >>>>> > >> > > > > > > > command for the suite is >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > mvn -DtestIgniteBinaryCacheQueryTestSuite -am -pl >> >>>>> > >> > > :ignite-indexing >> >>>>> > >> > > > > > > > surefire:test >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > Sequence of actions is: >> >>>>> > >> > > > > > > > 1. Find modules dependencies (*-am* flag): >> >>>>> ignite-tools, >> >>>>> > >> > > > ignite-core; >> >>>>> > >> > > > > > > > 2. Run the test command for every module. In this >> >>>>> step the >> >>>>> > >> > maven >> >>>>> > >> > > > > tries >> >>>>> > >> > > > > > to >> >>>>> > >> > > > > > > > find the specified test for every module. This is >> >>>>> good >> >>>>> > >> news, so >> >>>>> > >> > > we >> >>>>> > >> > > > > > don't >> >>>>> > >> > > > > > > > need to create new TeamCity suites for such >> splitted >> >>>>> > suites. >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > But the run performs within the current module >> >>>>> classpath, >> >>>>> > so >> >>>>> > >> > for >> >>>>> > >> > > > the >> >>>>> > >> > > > > > core >> >>>>> > >> > > > > > > > module the test suite fails with error "Add >> module >> >>>>> > >> > > > 'ignite-indexing' >> >>>>> > >> > > > > to >> >>>>> > >> > > > > > > the >> >>>>> > >> > > > > > > > classpath of all Ignite nodes". Maven can't >> >>>>> resolve it. >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > The only way to work with it is to specify >> >>>>> additional >> >>>>> > >> classpath >> >>>>> > >> > > > > > elements >> >>>>> > >> > > > > > > > for tests with setting >> >>>>> > >> > > > > > > *-Dmaven.test.additionalClasspath=/path/to/m2/jar*. >> >>>>> > >> > > > > > > > I did it by filling MAVEN_OPTS with the setting. >> >>>>> Please >> >>>>> > >> check >> >>>>> > >> > the >> >>>>> > >> > > > job >> >>>>> > >> > > > > > > > parameters [1]. After that the core module part >> ran >> >>>>> > >> > successfully. >> >>>>> > >> > > > It >> >>>>> > >> > > > > > > means >> >>>>> > >> > > > > > > > for every TC suite that runs such splitted suite >> we >> >>>>> need >> >>>>> > to >> >>>>> > >> set >> >>>>> > >> > > the >> >>>>> > >> > > > > > > > setting. What do you think, is it a valid way to >> >>>>> handle >> >>>>> > the >> >>>>> > >> > > issue? >> >>>>> > >> > > > If >> >>>>> > >> > > > > > > there >> >>>>> > >> > > > > > > > are no objections, I will check other such >> suites. >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > Also to mention there, the work directory >> contains a >> >>>>> > >> > > *repository/* >> >>>>> > >> > > > > > folder >> >>>>> > >> > > > > > > > with all required .jars. But usage of this path >> in >> >>>>> the >> >>>>> > >> setting >> >>>>> > >> > > > didn't >> >>>>> > >> > > > > > > help. >> >>>>> > >> > > > > > > > I'm not sure, but I think it's an issue due to >> >>>>> usage of >> >>>>> > >> > > > Classworlds. >> >>>>> > >> > > > > > So, >> >>>>> > >> > > > > > > > using dependency from .m2 is the only way. >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > [1] >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > >> >>>>> > >> > > > > > >> >>>>> > >> > > > > >> >>>>> > >> > > > >> >>>>> > >> > > >> >>>>> > >> > >> >>>>> > >> >> >>>>> > >> >>>>> >> https://ci.ignite.apache.org/viewLog.html?buildId=5770727&buildTypeId=IgniteTests24Java8_Queries1&tab=buildParameters >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > On Fri, Nov 27, 2020 at 3:55 PM Max Timonin < >> >>>>> > >> > > > timonin.ma...@gmail.com >> >>>>> > >> > > > > > >> >>>>> > >> > > > > > > > wrote: >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > > > Sure, I'll do that. >> >>>>> > >> > > > > > > > > >> >>>>> > >> > > > > > > > > On Fri, Nov 27, 2020 at 2:00 PM Ilya >> Kasnacheev < >> >>>>> > >> > > > > > > > ilya.kasnach...@gmail.com> >> >>>>> > >> > > > > > > > > wrote: >> >>>>> > >> > > > > > > > > >> >>>>> > >> > > > > > > > >> Hello! >> >>>>> > >> > > > > > > > >> >> >>>>> > >> > > > > > > > >> You can override these values (module, suites) >> >>>>> values >> >>>>> > >> when >> >>>>> > >> > > > > running a >> >>>>> > >> > > > > > > > suite >> >>>>> > >> > > > > > > > >> on TC. Can you please run these ones which >> need >> >>>>> to be >> >>>>> > >> > changed >> >>>>> > >> > > > > > > > individually >> >>>>> > >> > > > > > > > >> on TC, make sure they run without errors and >> >>>>> contain >> >>>>> > all >> >>>>> > >> the >> >>>>> > >> > > > > needed >> >>>>> > >> > > > > > > > tests, >> >>>>> > >> > > > > > > > >> and link to these runs in the ticket? Then I >> can >> >>>>> modify >> >>>>> > >> the >> >>>>> > >> > > > suites >> >>>>> > >> > > > > > to >> >>>>> > >> > > > > > > > fit >> >>>>> > >> > > > > > > > >> those. >> >>>>> > >> > > > > > > > >> >> >>>>> > >> > > > > > > > >> I'm not sure that class shadowing will work as >> >>>>> we want >> >>>>> > >> it to >> >>>>> > >> > > > work, >> >>>>> > >> > > > > > > e.g., >> >>>>> > >> > > > > > > > >> we >> >>>>> > >> > > > > > > > >> now have two IgniteCacheQuerySelfTestSuite6 >> with >> >>>>> the >> >>>>> > same >> >>>>> > >> > > FQDN, >> >>>>> > >> > > > > I'm >> >>>>> > >> > > > > > > not >> >>>>> > >> > > > > > > > >> sure if maven/TC is going to pick both or just >> >>>>> one. >> >>>>> > >> > > > > > > > >> Maybe they should go to a different package, >> >>>>> e.g., >> >>>>> > >> > > > testsuites/core >> >>>>> > >> > > > > > for >> >>>>> > >> > > > > > > > >> every suite already present in >> >>>>> indexing/spring/etc. >> >>>>> > Maybe >> >>>>> > >> > you >> >>>>> > >> > > > can >> >>>>> > >> > > > > > > rename >> >>>>> > >> > > > > > > > >> them just now? This will mean a lot less of >> work >> >>>>> > >> > reconfiguring >> >>>>> > >> > > > > > suites. >> >>>>> > >> > > > > > > > >> In TC configurations, suite names are simple >> >>>>> class >> >>>>> > names, >> >>>>> > >> > not >> >>>>> > >> > > > FQ, >> >>>>> > >> > > > > so >> >>>>> > >> > > > > > > no >> >>>>> > >> > > > > > > > >> changes may be needed at all. >> >>>>> > >> > > > > > > > >> >> >>>>> > >> > > > > > > > >> Regards, >> >>>>> > >> > > > > > > > >> -- >> >>>>> > >> > > > > > > > >> Ilya Kasnacheev >> >>>>> > >> > > > > > > > >> >> >>>>> > >> > > > > > > > >> >> >>>>> > >> > > > > > > > >> пт, 27 нояб. 2020 г. в 13:03, Max Timonin < >> >>>>> > >> > > > > timonin.ma...@gmail.com >> >>>>> > >> > > > > > >: >> >>>>> > >> > > > > > > > >> >> >>>>> > >> > > > > > > > >> > Hi, sorry for the misleading. I mean "adding >> >>>>> > >> ignite-core >> >>>>> > >> > > > module >> >>>>> > >> > > > > > > > >> *suites* to >> >>>>> > >> > > > > > > > >> > the TeamCity Queries* suite" >> >>>>> > >> > > > > > > > >> > >> >>>>> > >> > > > > > > > >> > On Fri, Nov 27, 2020 at 12:44 PM Ilya >> >>>>> Kasnacheev < >> >>>>> > >> > > > > > > > >> > ilya.kasnach...@gmail.com> >> >>>>> > >> > > > > > > > >> > wrote: >> >>>>> > >> > > > > > > > >> > >> >>>>> > >> > > > > > > > >> > > Hello! >> >>>>> > >> > > > > > > > >> > > >> >>>>> > >> > > > > > > > >> > > What do you mean by "adding ignite-core to >> >>>>> suite"? >> >>>>> > >> > > > ignite-core >> >>>>> > >> > > > > > is >> >>>>> > >> > > > > > > a >> >>>>> > >> > > > > > > > >> top >> >>>>> > >> > > > > > > > >> > > dependency and its tests are also included >> >>>>> in all >> >>>>> > >> other >> >>>>> > >> > > > > modules' >> >>>>> > >> > > > > > > > tests >> >>>>> > >> > > > > > > > >> > > classpath since it provides >> GridAbstractTest. >> >>>>> > >> > > > > > > > >> > > >> >>>>> > >> > > > > > > > >> > > Regards, >> >>>>> > >> > > > > > > > >> > > -- >> >>>>> > >> > > > > > > > >> > > Ilya Kasnacheev >> >>>>> > >> > > > > > > > >> > > >> >>>>> > >> > > > > > > > >> > > >> >>>>> > >> > > > > > > > >> > > пт, 27 нояб. 2020 г. в 01:24, Max Timonin >> < >> >>>>> > >> > > > > > > timonin.ma...@gmail.com >> >>>>> > >> > > > > > > > >: >> >>>>> > >> > > > > > > > >> > > >> >>>>> > >> > > > > > > > >> > > > Hi, Ilya! >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > So, I've updated PR, fixed comments and >> >>>>> removed >> >>>>> > >> Core* >> >>>>> > >> > > > > > prefixes. >> >>>>> > >> > > > > > > > >> MTCGA >> >>>>> > >> > > > > > > > >> > > shows >> >>>>> > >> > > > > > > > >> > > > no blockers, but it was 2 weeks ago, so >> >>>>> I've >> >>>>> > >> started >> >>>>> > >> > it >> >>>>> > >> > > > > again. >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > If PR is OK then there are some suites >> that >> >>>>> > should >> >>>>> > >> be >> >>>>> > >> > > > > updated >> >>>>> > >> > > > > > on >> >>>>> > >> > > > > > > > TC. >> >>>>> > >> > > > > > > > >> > > Could >> >>>>> > >> > > > > > > > >> > > > you please tell me how we can do it? >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 1. Add ignite-cassandra-serializers >> suite: >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 1. >> >>>>> org.apache.ignite.tests.SerializerSuite >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 2. Add ignite-core to Queries* TC suite: >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 1. >> >>>>> > >> org.apache.ignite.client.IgniteClientTestSuite >> >>>>> > >> > > > > > > > >> > > > 2. >> >>>>> > >> > > > > >> org.apache.ignite.suites.IgniteBinaryCacheQueryTestSuite >> >>>>> > >> > > > > > > > >> > > > 3. >> >>>>> > >> > > > > > >> >>>>> org.apache.ignite.suites.IgniteBinaryCacheQueryTestSuite2 >> >>>>> > >> > > > > > > > >> > > > 4. >> >>>>> > >> > > > > org.apache.ignite.suites.IgniteCacheQuerySelfTestSuite3 >> >>>>> > >> > > > > > > > >> > > > 5. >> >>>>> > >> > > > > org.apache.ignite.suites.IgniteCacheQuerySelfTestSuite4 >> >>>>> > >> > > > > > > > >> > > > 6. >> >>>>> > >> > > > > org.apache.ignite.suites.IgniteCacheQuerySelfTestSuite5 >> >>>>> > >> > > > > > > > >> > > > 7. >> >>>>> > >> > > > > org.apache.ignite.suites.IgniteCacheQuerySelfTestSuite6 >> >>>>> > >> > > > > > > > >> > > > 8. >> >>>>> > >> > > > > > > >> >>>>> org.apache.ignite.suites.IgnitePdsWithIndexingCoreTestSuite >> >>>>> > >> > > > > > > > >> > > > 9. >> >>>>> > >> > > org.apache.ignite.suites.IgniteCacheMvccSqlTestSuite >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 3. Remove ignite-indexing from TC >> suites: >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 1. >> >>>>> > >> > > > > > > >> >>>>> org.apache.ignite.testsuites.IgniteCacheQuerySelfTestSuite3 >> >>>>> > >> > > > > > > > >> > > > 2. >> >>>>> > >> > > > > > > >> >>>>> org.apache.ignite.testsuites.IgniteCacheQuerySelfTestSuite4 >> >>>>> > >> > > > > > > > >> > > > 3. >> >>>>> > >> > > > > > > >> >>>>> org.apache.ignite.testsuites.IgniteCacheQuerySelfTestSuite5 >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 4. Add ignite-core to Spring* TC suite: >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 1. >> >>>>> > >> > org.apache.ignite.testsuites.IgniteSpringTestSuite >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 5. Add ignite-core suite (depends on >> >>>>> > uri-deployment >> >>>>> > >> > > > module): >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 1. >> >>>>> > >> > > > > > >> >>>>> org.apache.ignite.testsuites.IgniteUriDeploymentTestSuite >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 6. Add ignite-core suite to Zookeeper TC >> >>>>> suite: >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 1. >> >>>>> > >> > > > > > > >> >>>>> org.apache.ignite.testsuites.ZookeeperDiscoverySpiTestSuite3 >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 7. Remove ignite-zookeeper test suite: >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 1. >> >>>>> > >> > > > > > > >> >>>>> org.apache.ignite.testsuites.ZookeeperDiscoverySpiTestSuite3 >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 8. Add ignite-ml test suites: >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > 1. >> >>>>> > >> > > > org.apache.ignite.ml.math.distances.DistancesTestSuite >> >>>>> > >> > > > > > > > >> > > > 2. org.apache.ignite.ml >> >>>>> > >> > > .naivebayes.NaiveBayesTestSuite >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > On Wed, Nov 25, 2020 at 4:26 PM Ilya >> >>>>> Kasnacheev < >> >>>>> > >> > > > > > > > >> > > ilya.kasnach...@gmail.com >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > wrote: >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > Hello! >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > Yes, we have such tests which depend >> on >> >>>>> > >> > > ignite-indexing >> >>>>> > >> > > > or >> >>>>> > >> > > > > > > > >> > > ignite-spring. >> >>>>> > >> > > > > > > > >> > > > > They just need to be included in >> Spring* >> >>>>> or >> >>>>> > >> Queries* >> >>>>> > >> > > > test >> >>>>> > >> > > > > > > suite. >> >>>>> > >> > > > > > > > >> Then >> >>>>> > >> > > > > > > > >> > > > they >> >>>>> > >> > > > > > > > >> > > > > will be executed on TC in the correct >> >>>>> context. >> >>>>> > >> You >> >>>>> > >> > can >> >>>>> > >> > > > > also >> >>>>> > >> > > > > > > run >> >>>>> > >> > > > > > > > >> these >> >>>>> > >> > > > > > > > >> > > > tests >> >>>>> > >> > > > > > > > >> > > > > from IDEA by specifying other module >> as >> >>>>> > >> classpath. >> >>>>> > >> > No >> >>>>> > >> > > > need >> >>>>> > >> > > > > > to >> >>>>> > >> > > > > > > > move >> >>>>> > >> > > > > > > > >> > the >> >>>>> > >> > > > > > > > >> > > > > classes around. >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > I will check the PR. >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > Regards, >> >>>>> > >> > > > > > > > >> > > > > -- >> >>>>> > >> > > > > > > > >> > > > > Ilya Kasnacheev >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > ср, 25 нояб. 2020 г. в 00:22, Max >> >>>>> Timonin < >> >>>>> > >> > > > > > > > >> timonin.ma...@gmail.com>: >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > > Ilya, Anton, Ivan, hi! >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > I fix some comments you leave in the >> >>>>> PR. >> >>>>> > Also I >> >>>>> > >> > > > checked >> >>>>> > >> > > > > > some >> >>>>> > >> > > > > > > > >> test >> >>>>> > >> > > > > > > > >> > > > suites >> >>>>> > >> > > > > > > > >> > > > > > and found that some tests are >> written >> >>>>> in the >> >>>>> > >> core >> >>>>> > >> > > > module >> >>>>> > >> > > > > > but >> >>>>> > >> > > > > > > > >> depend >> >>>>> > >> > > > > > > > >> > > on >> >>>>> > >> > > > > > > > >> > > > > the >> >>>>> > >> > > > > > > > >> > > > > > indexing module (or other modules). >> >>>>> Some of >> >>>>> > >> such >> >>>>> > >> > > test >> >>>>> > >> > > > > > > classes >> >>>>> > >> > > > > > > > >> > contain >> >>>>> > >> > > > > > > > >> > > > > tests >> >>>>> > >> > > > > > > > >> > > > > > that are related to the core >> >>>>> functionality, >> >>>>> > but >> >>>>> > >> > some >> >>>>> > >> > > > to >> >>>>> > >> > > > > > > > >> indexing. >> >>>>> > >> > > > > > > > >> > I'm >> >>>>> > >> > > > > > > > >> > > > not >> >>>>> > >> > > > > > > > >> > > > > > sure if it is correct to move a >> whole >> >>>>> suite >> >>>>> > >> with >> >>>>> > >> > all >> >>>>> > >> > > > > tests >> >>>>> > >> > > > > > > > from >> >>>>> > >> > > > > > > > >> the >> >>>>> > >> > > > > > > > >> > > > > > indexing module to the core, as it >> >>>>> will hide >> >>>>> > >> some >> >>>>> > >> > > core >> >>>>> > >> > > > > > tests >> >>>>> > >> > > > > > > > >> from >> >>>>> > >> > > > > > > > >> > the >> >>>>> > >> > > > > > > > >> > > > > core >> >>>>> > >> > > > > > > > >> > > > > > module. >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > I believe that the correct solution >> is >> >>>>> to >> >>>>> > >> > > investigate >> >>>>> > >> > > > > > every >> >>>>> > >> > > > > > > > such >> >>>>> > >> > > > > > > > >> > test >> >>>>> > >> > > > > > > > >> > > > and >> >>>>> > >> > > > > > > > >> > > > > > move it to the right module. But I >> >>>>> think this >> >>>>> > >> work >> >>>>> > >> > > > will >> >>>>> > >> > > > > > > take a >> >>>>> > >> > > > > > > > >> lot >> >>>>> > >> > > > > > > > >> > of >> >>>>> > >> > > > > > > > >> > > > > time >> >>>>> > >> > > > > > > > >> > > > > > and should be performed in a >> separate >> >>>>> > ticket, I >> >>>>> > >> > will >> >>>>> > >> > > > do >> >>>>> > >> > > > > it >> >>>>> > >> > > > > > > in >> >>>>> > >> > > > > > > > >> the >> >>>>> > >> > > > > > > > >> > > > > > background. >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > I think currently we should proceed >> >>>>> with a >> >>>>> > way >> >>>>> > >> I >> >>>>> > >> > > > > > introduced >> >>>>> > >> > > > > > > in >> >>>>> > >> > > > > > > > >> PR: >> >>>>> > >> > > > > > > > >> > > > > > 1. Create fake suites for all such >> >>>>> tests >> >>>>> > >> (written >> >>>>> > >> > in >> >>>>> > >> > > > > core, >> >>>>> > >> > > > > > > > >> suited >> >>>>> > >> > > > > > > > >> > in >> >>>>> > >> > > > > > > > >> > > > > other >> >>>>> > >> > > > > > > > >> > > > > > modules: >> >>>>> indexing/spring/zookeeper/etc) in >> >>>>> > the >> >>>>> > >> > core >> >>>>> > >> > > > > > module. >> >>>>> > >> > > > > > > I >> >>>>> > >> > > > > > > > >> named >> >>>>> > >> > > > > > > > >> > > > such >> >>>>> > >> > > > > > > > >> > > > > > suites with prefix Core*. >> >>>>> > >> > > > > > > > >> > > > > > 2. Replace tests in modules with >> links >> >>>>> to >> >>>>> > fake >> >>>>> > >> > > suites. >> >>>>> > >> > > > > > > > >> > > > > > 3. Create an umbrella Jira ticket to >> >>>>> discover >> >>>>> > >> > every >> >>>>> > >> > > > fake >> >>>>> > >> > > > > > > suite >> >>>>> > >> > > > > > > > >> and >> >>>>> > >> > > > > > > > >> > > > > replace >> >>>>> > >> > > > > > > > >> > > > > > it with a new one in the right >> module. >> >>>>> > >> > > > > > > > >> > > > > > 4. Merge this PR for introducing a >> new >> >>>>> travis >> >>>>> > >> > check >> >>>>> > >> > > to >> >>>>> > >> > > > > > avoid >> >>>>> > >> > > > > > > > >> losing >> >>>>> > >> > > > > > > > >> > > > > > new tests. >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > WDYT? >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > List of such mixed suites: >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > 1. suite >> >>>>> IgniteBinaryCacheQueryTestSuite >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > test >> >>>>> GridCacheQueryIndexingDisabledSelfTest >> >>>>> > >> > > > > > > > >> > > > > > test >> >>>>> IgniteCacheBinaryObjectsScanSelfTest >> >>>>> > >> > > > > > > > >> > > > > > test >> >>>>> > >> > IgniteCacheBinaryObjectsScanWithEventsSelfTest) >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > 2. suite >> IgniteCacheQuerySelfTestSuite3 >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > test >> >>>>> > GridCacheContinuousQueryNodesFilteringTest >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > 3. suite >> IgniteCacheQuerySelfTestSuite5 >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > test >> >>>>> > >> > > > > ContinuousQueryRemoteFilterMissingInClassPathSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > 4. suite >> IgniteCacheQuerySelfTestSuite6 >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > test >> >>>>> CacheContinuousQueryOperationP2PTest >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > test >> >>>>> > >> > CacheContinuousQueryFilterDeploymentFailedTest >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > 5. all tests in suite >> >>>>> > >> > > > IgnitePdsWithIndexingCoreTestSuite >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > 6. and some others. >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > On Wed, Nov 18, 2020 at 12:38 PM Max >> >>>>> Timonin >> >>>>> > < >> >>>>> > >> > > > > > > > >> > > timonin.ma...@gmail.com> >> >>>>> > >> > > > > > > > >> > > > > > wrote: >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > > > Hi Ilya! Thank you for up the >> topic. >> >>>>> I will >> >>>>> > >> come >> >>>>> > >> > > > back >> >>>>> > >> > > > > > with >> >>>>> > >> > > > > > > > >> fixes >> >>>>> > >> > > > > > > > >> > > and >> >>>>> > >> > > > > > > > >> > > > > > > comments in a couple of days. >> >>>>> > >> > > > > > > > >> > > > > > > >> >>>>> > >> > > > > > > > >> > > > > > > On Tue, Nov 17, 2020 at 4:26 PM >> Ilya >> >>>>> > >> Kasnacheev >> >>>>> > >> > < >> >>>>> > >> > > > > > > > >> > > > > > ilya.kasnach...@gmail.com> >> >>>>> > >> > > > > > > > >> > > > > > > wrote: >> >>>>> > >> > > > > > > > >> > > > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> Hello! >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> I have left some comments and >> >>>>> there's also >> >>>>> > >> more >> >>>>> > >> > > > > > > discussion >> >>>>> > >> > > > > > > > >> > there. >> >>>>> > >> > > > > > > > >> > > > Can >> >>>>> > >> > > > > > > > >> > > > > > you >> >>>>> > >> > > > > > > > >> > > > > > >> please look? >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> Thanks, >> >>>>> > >> > > > > > > > >> > > > > > >> -- >> >>>>> > >> > > > > > > > >> > > > > > >> Ilya Kasnacheev >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> вт, 3 нояб. 2020 г. в 00:03, Max >> >>>>> Timonin < >> >>>>> > >> > > > > > > > >> > timonin.ma...@gmail.com >> >>>>> > >> > > > > > > > >> > > >: >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > Hi! >> >>>>> > >> > > > > > > > >> > > > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > I've updated PR: >> >>>>> > >> > > > > > > > >> https://github.com/apache/ignite/pull/8367. >> >>>>> > >> > > > > > > > >> > > > Anton, >> >>>>> > >> > > > > > > > >> > > > > > >> Ivan, >> >>>>> > >> > > > > > > > >> > > > > > >> > Ivan could you please review >> it? >> >>>>> > >> > > > > > > > >> > > > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > Some moments to mention: >> >>>>> > >> > > > > > > > >> > > > > > >> > 1. I've added new suites: >> >>>>> > SerializerSuite >> >>>>> > >> > > > > > > > >> > > > > > >> (ignite-cassandra-serializers), >> >>>>> > >> > > > > > > > >> > > > > > >> > DistanceTestSuite, >> >>>>> NaiveBayesTestSuite >> >>>>> > >> > > > (ignite-ml). >> >>>>> > >> > > > > > > > Should >> >>>>> > >> > > > > > > > >> we >> >>>>> > >> > > > > > > > >> > > > > > configure >> >>>>> > >> > > > > > > > >> > > > > > >> a >> >>>>> > >> > > > > > > > >> > > > > > >> > TeamCity to run them? >> >>>>> > >> > > > > > > > >> > > > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > 2. Some tests marked as failed, >> >>>>> I'll >> >>>>> > >> create >> >>>>> > >> > > > > > > corresponding >> >>>>> > >> > > > > > > > >> > > tickets >> >>>>> > >> > > > > > > > >> > > > > for >> >>>>> > >> > > > > > > > >> > > > > > >> them >> >>>>> > >> > > > > > > > >> > > > > > >> > after PR approved: >> >>>>> > >> > > > > > > > >> > > > > > >> > - >> >>>>> IgnitePKIndexesMigrationToUnwrapPkTest >> >>>>> > >> > > > > > > > >> > > > > > >> > - P2PGridifySelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > - >> >>>>> > >> GridCacheMultithreadedFailoverAbstractTest >> >>>>> > >> > > > > > > > >> > > > > > >> > - WalCompactionAfterRestartTest >> >>>>> > >> > > > > > > > >> > > > > > >> > - >> GridTcpCommunicationSpiLogTest >> >>>>> > >> > > > > > > > >> > > > > > >> > - >> >>>>> ComplexSecondaryKeyUnwrapSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > - SqlTransactionsSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > 3. Add docs to DEVNOTES.txt >> >>>>> > >> > > > > > > > >> > > > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > On Mon, Nov 2, 2020 at 11:44 AM >> >>>>> Anton >> >>>>> > >> > > Vinogradov >> >>>>> > >> > > > < >> >>>>> > >> > > > > > > > >> > a...@apache.org >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > wrote: >> >>>>> > >> > > > > > > > >> > > > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > As I understand we >> >>>>> > >> > > > > > > > >> > > > > > >> > > > can't just move suites >> between >> >>>>> > >> modules, >> >>>>> > >> > as >> >>>>> > >> > > > > > TeamCity >> >>>>> > >> > > > > > > > may >> >>>>> > >> > > > > > > > >> > > depend >> >>>>> > >> > > > > > > > >> > > > > on >> >>>>> > >> > > > > > > > >> > > > > > >> the >> >>>>> > >> > > > > > > > >> > > > > > >> > > path >> >>>>> > >> > > > > > > > >> > > > > > >> > > > to them. >> >>>>> > >> > > > > > > > >> > > > > > >> > > See no problem to update TC >> as >> >>>>> well. >> >>>>> > >> > > > > > > > >> > > > > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > On Fri, Oct 30, 2020 at 4:32 >> PM >> >>>>> Ivan >> >>>>> > >> > > > Daschinsky < >> >>>>> > >> > > > > > > > >> > > > > > ivanda...@gmail.com> >> >>>>> > >> > > > > > > > >> > > > > > >> > > wrote: >> >>>>> > >> > > > > > > > >> > > > > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > I suggests to mark these >> >>>>> tests with >> >>>>> > >> > @Ignore >> >>>>> > >> > > > and >> >>>>> > >> > > > > > > file >> >>>>> > >> > > > > > > > >> > tickets >> >>>>> > >> > > > > > > > >> > > > to >> >>>>> > >> > > > > > > > >> > > > > > fix >> >>>>> > >> > > > > > > > >> > > > > > >> > them. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > пт, 30 окт. 2020 г. в >> 16:26, >> >>>>> Ivan >> >>>>> > >> > > Daschinsky >> >>>>> > >> > > > < >> >>>>> > >> > > > > > > > >> > > > > ivanda...@gmail.com >> >>>>> > >> > > > > > > > >> > > > > > >: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > Hi >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > >> >>>>> WalCompactionAfterRestartTest -- >> >>>>> > >> yes we >> >>>>> > >> > > > need >> >>>>> > >> > > > > > it. >> >>>>> > >> > > > > > > > This >> >>>>> > >> > > > > > > > >> > test >> >>>>> > >> > > > > > > > >> > > > > > failed >> >>>>> > >> > > > > > > > >> > > > > > >> > > because >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > of race (test shold be >> >>>>> rewritten a >> >>>>> > >> > little >> >>>>> > >> > > > > bit) >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > пт, 30 окт. 2020 г. в >> >>>>> 16:15, Max >> >>>>> > >> > Timonin >> >>>>> > >> > > < >> >>>>> > >> > > > > > > > >> > > > > > timonin.ma...@gmail.com >> >>>>> > >> > > > > > > > >> > > > > > >> >: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> Hi! >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> Yes, you're correct. >> I've >> >>>>> > developed >> >>>>> > >> > the >> >>>>> > >> > > > > check >> >>>>> > >> > > > > > > and >> >>>>> > >> > > > > > > > >> > started >> >>>>> > >> > > > > > > > >> > > > to >> >>>>> > >> > > > > > > > >> > > > > > >> clean >> >>>>> > >> > > > > > > > >> > > > > > >> > > tests >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> (move them to suites, >> mark >> >>>>> some >> >>>>> > >> tests >> >>>>> > >> > > with >> >>>>> > >> > > > > > > Ignore, >> >>>>> > >> > > > > > > > >> > > etc.). I >> >>>>> > >> > > > > > > > >> > > > > > >> finish >> >>>>> > >> > > > > > > > >> > > > > > >> > > work >> >>>>> > >> > > > > > > > >> > > > > > >> > > > on >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> the core module. I hope >> it >> >>>>> was >> >>>>> > the >> >>>>> > >> > > biggest >> >>>>> > >> > > > > > one, >> >>>>> > >> > > > > > > > and >> >>>>> > >> > > > > > > > >> > > others >> >>>>> > >> > > > > > > > >> > > > > are >> >>>>> > >> > > > > > > > >> > > > > > >> less. >> >>>>> > >> > > > > > > > >> > > > > > >> > > If >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> so, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> I think I will finish >> the >> >>>>> work on >> >>>>> > >> > other >> >>>>> > >> > > > > > modules >> >>>>> > >> > > > > > > > in 1 >> >>>>> > >> > > > > > > > >> > or 2 >> >>>>> > >> > > > > > > > >> > > > > > weeks, >> >>>>> > >> > > > > > > > >> > > > > > >> as >> >>>>> > >> > > > > > > > >> > > > > > >> > I >> >>>>> > >> > > > > > > > >> > > > > > >> > > do >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> this activity in the >> >>>>> background >> >>>>> > >> (~10% >> >>>>> > >> > of >> >>>>> > >> > > > my >> >>>>> > >> > > > > > work >> >>>>> > >> > > > > > > > >> time). >> >>>>> > >> > > > > > > > >> > > > > > Actually >> >>>>> > >> > > > > > > > >> > > > > > >> > I've >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> found >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> 3 failed tests in the >> core >> >>>>> module >> >>>>> > >> that >> >>>>> > >> > > > > aren't >> >>>>> > >> > > > > > in >> >>>>> > >> > > > > > > > any >> >>>>> > >> > > > > > > > >> > > suite, >> >>>>> > >> > > > > > > > >> > > > > so >> >>>>> > >> > > > > > > > >> > > > > > I >> >>>>> > >> > > > > > > > >> > > > > > >> > need >> >>>>> > >> > > > > > > > >> > > > > > >> > > > time >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> to discover reason of >> >>>>> failures >> >>>>> > and >> >>>>> > >> if >> >>>>> > >> > we >> >>>>> > >> > > > > > > actually >> >>>>> > >> > > > > > > > >> need >> >>>>> > >> > > > > > > > >> > > > those >> >>>>> > >> > > > > > > > >> > > > > > >> tests: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> > >> > > GridCacheMultithreadedFailoverAbstractTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> WalCompactionAfterRestartTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> GridTcpCommunicationSpiLogTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> Also we should decide >> how >> >>>>> to be >> >>>>> > >> with >> >>>>> > >> > > > wrongly >> >>>>> > >> > > > > > > > located >> >>>>> > >> > > > > > > > >> > es. >> >>>>> > >> > > > > > > > >> > > > As I >> >>>>> > >> > > > > > > > >> > > > > > >> > > understand >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> we >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> can't just move suites >> >>>>> between >> >>>>> > >> > modules, >> >>>>> > >> > > as >> >>>>> > >> > > > > > > > TeamCity >> >>>>> > >> > > > > > > > >> may >> >>>>> > >> > > > > > > > >> > > > > depend >> >>>>> > >> > > > > > > > >> > > > > > on >> >>>>> > >> > > > > > > > >> > > > > > >> > the >> >>>>> > >> > > > > > > > >> > > > > > >> > > > path >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> to them. So, for such >> >>>>> cases I've >> >>>>> > >> just >> >>>>> > >> > > > > created >> >>>>> > >> > > > > > > > >> suites in >> >>>>> > >> > > > > > > > >> > > the >> >>>>> > >> > > > > > > > >> > > > > > right >> >>>>> > >> > > > > > > > >> > > > > > >> > > > module, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> and replaced the test >> list >> >>>>> with >> >>>>> > the >> >>>>> > >> > new >> >>>>> > >> > > > > class >> >>>>> > >> > > > > > > > >> suite. It >> >>>>> > >> > > > > > > > >> > > > does >> >>>>> > >> > > > > > > > >> > > > > > not >> >>>>> > >> > > > > > > > >> > > > > > >> > look >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> pretty enough, but I >> think >> >>>>> It's a >> >>>>> > >> path >> >>>>> > >> > > of >> >>>>> > >> > > > > > least >> >>>>> > >> > > > > > > > >> > > resistance. >> >>>>> > >> > > > > > > > >> > > > > > WDYT? >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> BEFORE: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> Module A -> SuiteA -> >> >>>>> testA1, >> >>>>> > >> testA2, >> >>>>> > >> > > > > testB1, >> >>>>> > >> > > > > > > > testB2 >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> Module B -> testB1, >> testB2 >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> AFTER: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> Module A -> SuiteA, >> SuiteB >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> Module B -> SuiteB -> >> >>>>> testB1, >> >>>>> > >> testB2 >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> On Fri, Oct 30, 2020 at >> >>>>> 3:38 PM >> >>>>> > >> Anton >> >>>>> > >> > > > > > > Vinogradov < >> >>>>> > >> > > > > > > > >> > > > > > a...@apache.org> >> >>>>> > >> > > > > > > > >> > > > > > >> > > wrote: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > Folks, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > What's the current >> state >> >>>>> of >> >>>>> > this >> >>>>> > >> > > thread? >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > AFAIU, we found unused >> >>>>> and >> >>>>> > >> wrongly >> >>>>> > >> > > > located >> >>>>> > >> > > > > > > tests >> >>>>> > >> > > > > > > > >> and >> >>>>> > >> > > > > > > > >> > > > > > developed >> >>>>> > >> > > > > > > > >> > > > > > >> > some >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > checker, could we >> split >> >>>>> this to >> >>>>> > >> some >> >>>>> > >> > > > PRs? >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > Let's merge tests >> usage >> >>>>> fix and >> >>>>> > >> > > location >> >>>>> > >> > > > > > fixes >> >>>>> > >> > > > > > > > >> first, >> >>>>> > >> > > > > > > > >> > > > this >> >>>>> > >> > > > > > > > >> > > > > > will >> >>>>> > >> > > > > > > > >> > > > > > >> > > > provide >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> us >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > an ability to automate >> >>>>> check >> >>>>> > >> using >> >>>>> > >> > > > Travis. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > On Tue, Oct 20, 2020 >> at >> >>>>> 12:06 >> >>>>> > PM >> >>>>> > >> > Ivan >> >>>>> > >> > > > > > > Pavlukhin >> >>>>> > >> > > > > > > > < >> >>>>> > >> > > > > > > > >> > > > > > >> > > vololo...@gmail.com> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > wrote: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > Max, Ivan, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > Using explicit >> @Ignore >> >>>>> and >> >>>>> > the >> >>>>> > >> > > > automated >> >>>>> > >> > > > > > > check >> >>>>> > >> > > > > > > > >> > sounds >> >>>>> > >> > > > > > > > >> > > > > good >> >>>>> > >> > > > > > > > >> > > > > > to >> >>>>> > >> > > > > > > > >> > > > > > >> > me. >> >>>>> > >> > > > > > > > >> > > > > > >> > > If >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > nobody has arguments >> >>>>> against >> >>>>> > >> it I >> >>>>> > >> > > > think >> >>>>> > >> > > > > we >> >>>>> > >> > > > > > > > >> should >> >>>>> > >> > > > > > > > >> > do >> >>>>> > >> > > > > > > > >> > > > it. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > 2020-10-19 19:30 >> >>>>> GMT+03:00, >> >>>>> > Max >> >>>>> > >> > > > Timonin >> >>>>> > >> > > > > < >> >>>>> > >> > > > > > > > >> > > > > > >> > timonin.ma...@gmail.com >> >>>>> > >> > > > > > > > >> > > > > > >> > > >: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > Hi Ivan, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > I've checked the >> >>>>> ticket you >> >>>>> > >> > > provide. >> >>>>> > >> > > > > It >> >>>>> > >> > > > > > > > >> contains >> >>>>> > >> > > > > > > > >> > > > > subtasks >> >>>>> > >> > > > > > > > >> > > > > > >> to >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> uncomment >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > or >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > to remove some >> unused >> >>>>> > tests. >> >>>>> > >> It >> >>>>> > >> > > > > > definitely >> >>>>> > >> > > > > > > > >> > > describes >> >>>>> > >> > > > > > > > >> > > > > some >> >>>>> > >> > > > > > > > >> > > > > > >> > cases >> >>>>> > >> > > > > > > > >> > > > > > >> > > > I've >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > found. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > So what do you >> think >> >>>>> if I >> >>>>> > >> > > uncomment >> >>>>> > >> > > > > them >> >>>>> > >> > > > > > > in >> >>>>> > >> > > > > > > > >> > suites, >> >>>>> > >> > > > > > > > >> > > > add >> >>>>> > >> > > > > > > > >> > > > > > >> > @Ignore >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > annotation >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > for those tests >> >>>>> while the >> >>>>> > >> > tickets >> >>>>> > >> > > > are >> >>>>> > >> > > > > > > open? >> >>>>> > >> > > > > > > > >> This >> >>>>> > >> > > > > > > > >> > > will >> >>>>> > >> > > > > > > > >> > > > > > help >> >>>>> > >> > > > > > > > >> > > > > > >> to >> >>>>> > >> > > > > > > > >> > > > > > >> > > find >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> out >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > tests that were >> >>>>> forgiven >> >>>>> > in a >> >>>>> > >> > > recent >> >>>>> > >> > > > > > time. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > Also I believe >> that >> >>>>> this >> >>>>> > >> check >> >>>>> > >> > > must >> >>>>> > >> > > > be >> >>>>> > >> > > > > > > > >> > automated. I >> >>>>> > >> > > > > > > > >> > > > > > didn't >> >>>>> > >> > > > > > > > >> > > > > > >> > find >> >>>>> > >> > > > > > > > >> > > > > > >> > > a >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> way >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > how >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > uncomment / unused >> >>>>> tests >> >>>>> > are >> >>>>> > >> > found >> >>>>> > >> > > > in >> >>>>> > >> > > > > > the >> >>>>> > >> > > > > > > > >> ticket. >> >>>>> > >> > > > > > > > >> > > If >> >>>>> > >> > > > > > > > >> > > > > > there >> >>>>> > >> > > > > > > > >> > > > > > >> is >> >>>>> > >> > > > > > > > >> > > > > > >> > no >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> any - >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > I >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > propose mine PR >> for >> >>>>> this >> >>>>> > >> > purpose. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > On Mon, Oct 19, >> 2020 >> >>>>> at >> >>>>> > 5:24 >> >>>>> > >> PM >> >>>>> > >> > > Ivan >> >>>>> > >> > > > > > > > >> Daschinsky < >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> ivanda...@gmail.com> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > wrote: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> Ivan, as far as I >> >>>>> > >> understand, >> >>>>> > >> > Max >> >>>>> > >> > > > > also >> >>>>> > >> > > > > > > > >> created >> >>>>> > >> > > > > > > > >> > > > > > >> verification >> >>>>> > >> > > > > > > > >> > > > > > >> > > check >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> for >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > not >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> included test and >> >>>>> found a >> >>>>> > >> few >> >>>>> > >> > > > tests, >> >>>>> > >> > > > > > that >> >>>>> > >> > > > > > > > >> have >> >>>>> > >> > > > > > > > >> > > never >> >>>>> > >> > > > > > > > >> > > > > > been >> >>>>> > >> > > > > > > > >> > > > > > >> > > > included >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> in >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > any >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> testsuites. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> Also, I suppose, >> >>>>> that even >> >>>>> > >> if >> >>>>> > >> > we >> >>>>> > >> > > > > cannot >> >>>>> > >> > > > > > > run >> >>>>> > >> > > > > > > > >> some >> >>>>> > >> > > > > > > > >> > > > > tests, >> >>>>> > >> > > > > > > > >> > > > > > >> these >> >>>>> > >> > > > > > > > >> > > > > > >> > > > tests >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> should >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> be ignored using >> >>>>> > annotation, >> >>>>> > >> > but >> >>>>> > >> > > > not >> >>>>> > >> > > > > > > > >> commented. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> пн, 19 окт. 2020 >> г. >> >>>>> в >> >>>>> > 16:33, >> >>>>> > >> > Ivan >> >>>>> > >> > > > > > > > Pavlukhin < >> >>>>> > >> > > > > > > > >> > > > > > >> > > vololo...@gmail.com >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > Hi Max, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > There is an >> >>>>> existing >> >>>>> > >> effort >> >>>>> > >> > > about >> >>>>> > >> > > > > > > > >> "abandoned" >> >>>>> > >> > > > > > > > >> > > > tests >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > >> >>>>> > >> > > > > > > > >> > > >> >>>>> https://issues.apache.org/jira/browse/IGNITE-9210 >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > 2020-10-19 >> 16:25 >> >>>>> > >> GMT+03:00, >> >>>>> > >> > Max >> >>>>> > >> > > > > > > Timonin < >> >>>>> > >> > > > > > > > >> > > > > > >> > > > timonin.ma...@gmail.com >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > Hi Igniters! >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > I made a >> >>>>> research into >> >>>>> > >> > tests >> >>>>> > >> > > > that >> >>>>> > >> > > > > > > > aren't >> >>>>> > >> > > > > > > > >> > > > included >> >>>>> > >> > > > > > > > >> > > > > in >> >>>>> > >> > > > > > > > >> > > > > > >> any >> >>>>> > >> > > > > > > > >> > > > > > >> > > test >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > suite. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > As >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > TeamCity runs >> >>>>> tests by >> >>>>> > >> > suites >> >>>>> > >> > > > so >> >>>>> > >> > > > > > > there >> >>>>> > >> > > > > > > > >> could >> >>>>> > >> > > > > > > > >> > > be >> >>>>> > >> > > > > > > > >> > > > > > tests >> >>>>> > >> > > > > > > > >> > > > > > >> > that >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> never >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > run >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > on >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > TC. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > So I tried >> >>>>> > implementing >> >>>>> > >> a >> >>>>> > >> > > > simple >> >>>>> > >> > > > > > > check >> >>>>> > >> > > > > > > > >> for >> >>>>> > >> > > > > > > > >> > > such >> >>>>> > >> > > > > > > > >> > > > > > tests >> >>>>> > >> > > > > > > > >> > > > > > >> and >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> include >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > it >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > in >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > Ignite's >> travis >> >>>>> > config. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > The check >> runs >> >>>>> while >> >>>>> > >> "mvn >> >>>>> > >> > > test" >> >>>>> > >> > > > > > > command >> >>>>> > >> > > > > > > > >> and >> >>>>> > >> > > > > > > > >> > > > > > >> piggy-backs >> >>>>> > >> > > > > > > > >> > > > > > >> > on >> >>>>> > >> > > > > > > > >> > > > > > >> > > > the >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > maven >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > surefire >> >>>>> plugin. I >> >>>>> > >> replaced >> >>>>> > >> > > the >> >>>>> > >> > > > > > junit >> >>>>> > >> > > > > > > > >> > provider >> >>>>> > >> > > > > > > > >> > > > > with >> >>>>> > >> > > > > > > > >> > > > > > a >> >>>>> > >> > > > > > > > >> > > > > > >> > > custom >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> one >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > that >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > checks if a >> >>>>> class is a >> >>>>> > >> test >> >>>>> > >> > > or >> >>>>> > >> > > > a >> >>>>> > >> > > > > > > suite >> >>>>> > >> > > > > > > > >> > (there >> >>>>> > >> > > > > > > > >> > > > are >> >>>>> > >> > > > > > > > >> > > > > > some >> >>>>> > >> > > > > > > > >> > > > > > >> > > Ignite >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > specific >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > stuff), marks >> >>>>> tests >> >>>>> > that >> >>>>> > >> > are >> >>>>> > >> > > in >> >>>>> > >> > > > > > > suites >> >>>>> > >> > > > > > > > >> and >> >>>>> > >> > > > > > > > >> > > > raises >> >>>>> > >> > > > > > > > >> > > > > an >> >>>>> > >> > > > > > > > >> > > > > > >> > > > exception >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> if >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > there >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > are >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > non-suited >> >>>>> tests. It's >> >>>>> > >> > > > > implemented >> >>>>> > >> > > > > > > as a >> >>>>> > >> > > > > > > > >> part >> >>>>> > >> > > > > > > > >> > > of >> >>>>> > >> > > > > > > > >> > > > > > maven >> >>>>> > >> > > > > > > > >> > > > > > >> > > command >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> so >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > it >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> runs >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > for every >> module >> >>>>> > >> > separately. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > I've prepared >> >>>>> draft PR >> >>>>> > >> with >> >>>>> > >> > > > this >> >>>>> > >> > > > > > > check: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> https://github.com/apache/ignite/pull/8367 >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > Travis check >> >>>>> report is >> >>>>> > >> > here: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > >> >>>>> > >> > > > > > > > >> >>>>> https://travis-ci.org/github/apache/ignite/jobs/737046387 >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > As It's a >> >>>>> draft, so I >> >>>>> > >> skip >> >>>>> > >> > > some >> >>>>> > >> > > > > > maven >> >>>>> > >> > > > > > > > >> > > > > configuration >> >>>>> > >> > > > > > > > >> > > > > > >> steps >> >>>>> > >> > > > > > > > >> > > > > > >> > > > for a >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > while. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > Also >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > I run the >> check >> >>>>> only >> >>>>> > for >> >>>>> > >> > the >> >>>>> > >> > > > core >> >>>>> > >> > > > > > > > module. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > But I have >> some >> >>>>> > results >> >>>>> > >> > that >> >>>>> > >> > > > want >> >>>>> > >> > > > > > to >> >>>>> > >> > > > > > > > >> discuss >> >>>>> > >> > > > > > > > >> > > > > before >> >>>>> > >> > > > > > > > >> > > > > > >> > > continue >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> the >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > work: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > 1. Currently >> in >> >>>>> the >> >>>>> > core >> >>>>> > >> > > module >> >>>>> > >> > > > > > there >> >>>>> > >> > > > > > > > >> are 53 >> >>>>> > >> > > > > > > > >> > > > tests >> >>>>> > >> > > > > > > > >> > > > > > >> that >> >>>>> > >> > > > > > > > >> > > > > > >> > > > aren't >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > part >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > of >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > any >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > test suite. >> I'm >> >>>>> not >> >>>>> > sure >> >>>>> > >> > > about >> >>>>> > >> > > > > the >> >>>>> > >> > > > > > > > reason >> >>>>> > >> > > > > > > > >> > for >> >>>>> > >> > > > > > > > >> > > > > every >> >>>>> > >> > > > > > > > >> > > > > > >> test. >> >>>>> > >> > > > > > > > >> > > > > > >> > > So >> >>>>> > >> > > > > > > > >> > > > > > >> > > > I >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > just >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > put >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > below a list >> of >> >>>>> the >> >>>>> > >> tests >> >>>>> > >> > and >> >>>>> > >> > > > > last >> >>>>> > >> > > > > > > > >> > contributor >> >>>>> > >> > > > > > > > >> > > > to >> >>>>> > >> > > > > > > > >> > > > > a >> >>>>> > >> > > > > > > > >> > > > > > >> file >> >>>>> > >> > > > > > > > >> > > > > > >> > > that >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > contains >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> a >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > test. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > 2. Some tests >> >>>>> are >> >>>>> > >> located >> >>>>> > >> > in >> >>>>> > >> > > > the >> >>>>> > >> > > > > > core >> >>>>> > >> > > > > > > > >> > module, >> >>>>> > >> > > > > > > > >> > > > but >> >>>>> > >> > > > > > > > >> > > > > > >> suites >> >>>>> > >> > > > > > > > >> > > > > > >> > > are >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> in a >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > different, >> for >> >>>>> example >> >>>>> > >> > > > > > > ignite-indexing >> >>>>> > >> > > > > > > > >> suite >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > IgniteCacheQuerySelfTestSuite3 >> >>>>> > >> > > > > > > contains >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > only tests >> >>>>> written in >> >>>>> > >> the >> >>>>> > >> > > core >> >>>>> > >> > > > > > > module, >> >>>>> > >> > > > > > > > >> and >> >>>>> > >> > > > > > > > >> > > none >> >>>>> > >> > > > > > > > >> > > > > from >> >>>>> > >> > > > > > > > >> > > > > > >> the >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> indexing >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> module. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > Also there >> are >> >>>>> suites >> >>>>> > in >> >>>>> > >> > > > spring, >> >>>>> > >> > > > > > > > >> uri-deploy, >> >>>>> > >> > > > > > > > >> > > > > > zookeeper >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> modules. In >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > my >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> PR >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > I've just >> >>>>> copied the >> >>>>> > >> test >> >>>>> > >> > > > suites >> >>>>> > >> > > > > to >> >>>>> > >> > > > > > > the >> >>>>> > >> > > > > > > > >> core >> >>>>> > >> > > > > > > > >> > > > > module. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > 3. Some test >> >>>>> classes >> >>>>> > are >> >>>>> > >> > > named >> >>>>> > >> > > > > with >> >>>>> > >> > > > > > > the >> >>>>> > >> > > > > > > > >> > > > "Abstract" >> >>>>> > >> > > > > > > > >> > > > > > >> suffix >> >>>>> > >> > > > > > > > >> > > > > > >> > > but >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > don't >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> have >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > the >> >>>>> corresponding >> >>>>> > >> modifier >> >>>>> > >> > > (for >> >>>>> > >> > > > > > > > example, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > IgniteTxTimeoutAbstractTest). >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > So, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > I add the >> >>>>> modifier for >> >>>>> > >> > every >> >>>>> > >> > > > such >> >>>>> > >> > > > > > > file >> >>>>> > >> > > > > > > > if >> >>>>> > >> > > > > > > > >> > it's >> >>>>> > >> > > > > > > > >> > > > > not a >> >>>>> > >> > > > > > > > >> > > > > > >> part >> >>>>> > >> > > > > > > > >> > > > > > >> > > of >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> any >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > suite. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > What do you >> >>>>> think >> >>>>> > about >> >>>>> > >> > this >> >>>>> > >> > > > > check? >> >>>>> > >> > > > > > > If >> >>>>> > >> > > > > > > > >> > Ignite >> >>>>> > >> > > > > > > > >> > > > > needs >> >>>>> > >> > > > > > > > >> > > > > > >> it, >> >>>>> > >> > > > > > > > >> > > > > > >> > > let's >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > discuss >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > next >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > things: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > 1. Mark tests >> >>>>> that >> >>>>> > >> should >> >>>>> > >> > > never >> >>>>> > >> > > > > be >> >>>>> > >> > > > > > in >> >>>>> > >> > > > > > > > any >> >>>>> > >> > > > > > > > >> > > suite >> >>>>> > >> > > > > > > > >> > > > by >> >>>>> > >> > > > > > > > >> > > > > > >> some >> >>>>> > >> > > > > > > > >> > > > > > >> > > > reason; >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > 2. Fix the >> >>>>> missed >> >>>>> > tests; >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > 3. How to >> >>>>> declare >> >>>>> > suites >> >>>>> > >> > that >> >>>>> > >> > > > > > > contains >> >>>>> > >> > > > > > > > >> tests >> >>>>> > >> > > > > > > > >> > > > from >> >>>>> > >> > > > > > > > >> > > > > a >> >>>>> > >> > > > > > > > >> > > > > > >> > > different >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > module; >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > 4. How to >> check >> >>>>> if TC >> >>>>> > >> runs >> >>>>> > >> > > all >> >>>>> > >> > > > > > > suites. >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > List of >> >>>>> non-suited >> >>>>> > >> tests in >> >>>>> > >> > > the >> >>>>> > >> > > > > > core >> >>>>> > >> > > > > > > > >> module: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> maksim.stepac...@gmail.com >> >>>>> > >> > : >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > GridTcpCommunicationSpiLogTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> nizhi...@apache.org: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > IgniteCacheClientMultiNodeUpdateTopologyLockTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > CacheClientsConcurrentStartTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > IgniteOutOfMemoryPropagationTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > GridCacheP2PUndeploySelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > GridCacheRebalancingOrderingTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > IgniteMassLoadSandboxTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > PageLockTrackerMXBeanImplTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > >> IgniteBinaryMetadataUpdateNodeRestartTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > CacheLockCandidatesThreadTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > GridMBeanBaselineTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > >> RendezvousAffinityFunctionSimpleBenchmark >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> samvi...@yandex.ru: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > IgnitePdsNoSpaceLeftOnDeviceTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> maxmu...@gmail.com: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > GridCacheOnCopyFlagReplicatedSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > GridCacheOnCopyFlagLocalSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > GridCacheReplicatedAtomicReferenceMultiNodeTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> GridCacheReplicatedMarshallerTxTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > GridCacheReplicatedTxConcurrentGetTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > GridCacheOnCopyFlagTxPartitionedSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > GridCacheReplicatedTxReadTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > GridCachePartitionedAtomicReferenceMultiNodeTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > GridCacheOnCopyFlagAtomicSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> mmu...@apache.org: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > GridActivateExtensionTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > IgniteChangeGlobalStateCacheTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > IgniteChangeGlobalStateTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> IgniteChangeGlobalStateServiceTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > IgniteChangeGlobalStateDataStructureTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > oignate...@gridgain.com >> >>>>> > >> : >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > CacheEntryProcessorCopySelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > MemoryLeaksOnRestartNodeTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > GridCacheAtomicPreloadSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > WalCompactionAfterRestartTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > IgniteCacheConcurrentPutGetRemove >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > GridIoManagerBenchmark0 >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> nsamelc...@gmail.com: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > GridLongRunningInitNewCrdFutureDiagnosticsTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > >> GridCacheMultithreadedFailoverAbstractTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> alexey.goncha...@gmail.com >> >>>>> > >> > : >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > >> GridCacheBinaryObjectsAtomicOnheapSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >>>>> > >> > > > > > > > >> >>>>> GridCacheBinaryObjectsAtomicNearDisabledOnheapSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > GridCacheBinaryObjectsPartitionedOnheapSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> > >> > > > > >> >>>>> GridCacheBinaryObjectsPartitionedNearDisabledOnheapSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> vladis...@gmail.com: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > IgnitePartitionedLockSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> alexandr.bel...@xored.com: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> IgniteStableBaselineCachePutAllFailoverTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> IgniteStableBaselineCacheRemoveFailoverTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> ilant...@gridgain.com >> >>>>> > : >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > IgniteCacheAtomicOnheapExpiryPolicyTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> IgniteCacheAtomicLocalOnheapExpiryPolicyTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > GridCacheReplicatedOnheapFullApiSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > >> GridCacheBinaryObjectsLocalOnheapSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > oignate...@users.noreply.github.com: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> GridCacheTtlManagerEvictionSelfTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> ira...@apache.org: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> CommonPoolStarvationCheckpointTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> alievmi...@gmail.com: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > RemoveAllDeadlockTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > schugu...@gridgain.com: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> FullyConnectedComponentSearcherTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> sboi...@gridgain.com: >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > IgniteDataStructuresNoClassOnServerTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > timonin.ma...@gmail.com >> >>>>> > >> : >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> ReliableChannelTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > >> ThinClientPartitionAwarenessDiscoveryTest >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > -- >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > Best regards, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > Ivan Pavlukhin >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> -- >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> Sincerely yours, >> >>>>> Ivan >> >>>>> > >> > Daschinskiy >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > -- >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > Best regards, >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > Ivan Pavlukhin >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > -- >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > Sincerely yours, Ivan >> >>>>> Daschinskiy >> >>>>> > >> > > > > > > > >> > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > > -- >> >>>>> > >> > > > > > > > >> > > > > > >> > > > Sincerely yours, Ivan >> >>>>> Daschinskiy >> >>>>> > >> > > > > > > > >> > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > > > > >> > > >> >>>>> > >> > > > > > > > >> > > > > > >> > >> >>>>> > >> > > > > > > > >> > > > > > >> >> >>>>> > >> > > > > > > > >> > > > > > > >> >>>>> > >> > > > > > > > >> > > > > > >> >>>>> > >> > > > > > > > >> > > > > >> >>>>> > >> > > > > > > > >> > > > >> >>>>> > >> > > > > > > > >> > > >> >>>>> > >> > > > > > > > >> > >> >>>>> > >> > > > > > > > >> >> >>>>> > >> > > > > > > > > >> >>>>> > >> > > > > > > > >> >>>>> > >> > > > > > > >> >>>>> > >> > > > > > >> >>>>> > >> > > > > >> >>>>> > >> > > > >> >>>>> > >> > > > >> >>>>> > >> > > > -- >> >>>>> > >> > > > Sincerely yours, Ivan Daschinskiy >> >>>>> > >> > > > >> >>>>> > >> > > >> >>>>> > >> > >> >>>>> > >> >> >>>>> > > >> >>>>> > >> >>>>> >> >>>> >> >>> >> >> >> > >
