Hi Petr! My proposal is: 1. Create a parameter in [Build] TC suite - MAVEN_CHECKS, default value is "-Plicenses,checkstyle,check-licenses,check-test-suites". 2. Use it in a command along with MAVEN_MODULES_STRING. -U -Pall-java,all-scala,scala,lgpl,examples %MAVEN_CHECKS% %MAVEN_MODULES_STRING%
3. Provide a global param for test suites "reverse.dep.MAVEN_CHECKS" that is possible to override in a custom build. If I understand it correctly is possible to do by editing the job [1]. 4. This param should be represented to a user as a selector with 2 options: - default (see point 1.) - "-DskipTests=true" - that ignores all checks, skip tests and just build a .zip of Ignite. Could you please review this solution? Is it OK for you? [1] https://ci.ignite.apache.org/admin/editBuildParams.html?id=template:IgniteTests24Java8_RunTestSuitesJava On Thu, Feb 25, 2021 at 1:47 PM Petr Ivanov <piva...@gridgain.com> wrote: > If profile can handle this — its ok. > > For choosing build type — we can introduce select, that will choose > between -p <profile> and -DskipTests=true (defaulting to profile). > Thus [Build] will pass either way. > > > Regards, > Petr Ivanov > Head of IT > IT & Development Solutions | GRIDGAIN SYSTEMS > +7 (911) 945-00-59 > > > On 25 Feb 2021, at 13:23, Max Timonin <timonin.ma...@gmail.com> wrote: > > > > Yes, it's correct that "mvn install" runs also the "mvn test" command, > and > > this is OK as the check-test-suites profile handles all tests > > without running them. If the skipTests flag is triggered then this check > is > > useless. It will take only about 2 min to run "mvn test" with this > profile. > > Travis does that as one of steps. > > > > So, there are no issues with tests. Should I provide more info how this > > check works? > > > > Also, discussed with Anton Vinogradov, Alex Plekhanov. There can be an > > issue, that sometimes it's required to run custom test suites to debug > > flaky tests. Sequence of steps is the following: > > 1. Find a test suite with flaky tests (that reproducible only on an > > TeamCity agent); > > 2. Comment some tests in the suite to isolate; > > 3. Push it, and run related TC suite; > > 4. TC suite depends on [Build] job, run the job - it will fail on the > check > > "check-test-suites". > > > > So it is needed to provide a configuration to disable this check such > runs. > > I'll have a look on next week how to implement this. > > > > On Thu, Feb 25, 2021 at 11:02 AM Petr Ivanov <piva...@gridgain.com> > wrote: > > > >> I am telling that INSTALL goal for maven will trigger TEST goal for the > >> whole project and it cannot be prevented until the flag is specified > either > >> as command line parameter, or in profile somehow to be inherited by > other > >> modules. > >> Thats why I am suggesting this as separate suite. > >> > >> > >> Regards, > >> *Petr Ivanov* > >> Head of IT > >> IT & Development Solutions | > >> *GRIDGAIN SYSTEMS*+7 (911) 945-00-59 > >> > >> On 25 Feb 2021, at 10:44, Max Timonin <timonin.ma...@gmail.com> wrote: > >> > >> 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 > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>> > >>>>>>>>> > >>>>>>>> > >>>>>>> > >>>>>> > >>>> > >>> > >>> > >> > >