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