Max, > There is an overhead for running git + mvn test twice, but it is a cost for > the flexibility.
Yes, I mean this issue. I have no objections, the build queue seems to be empty most of the time. On Wed, 3 Mar 2021 at 21:01, Max Timonin <timonin.ma...@gmail.com> wrote: > > Yes, they can be performed as parallel, as doesn't depend on each other. > There is an overhead for running git + mvn test twice, but it is a cost for > the flexibility. > > On Wed, Mar 3, 2021 at 8:55 PM Max Timonin <timonin.ma...@gmail.com> wrote: > > > I mean that any TC job with tests depends on both [Build], [Sanity > > Checks]. No tests run if any of those jobs failed. > > > > [Build] prepares ignite.zip for distribution between TC agents (mvn > > install). > > [Sanity Checks] checks that code is correct in terms of our static checks > > (mvn test). > > > > Indeed it can be run as a single job, but in favor of flexibility in > > configuration (enable / disable checks) it is OK to separate it in 2 steps. > > > > Do you have some objections to do it that way? > > > > On Wed, Mar 3, 2021 at 8:45 PM Maxim Muzafarov <mmu...@apache.org> wrote: > > > >> Maxim, > >> > >> Can you clarify what means '[Sanity Checks] runs in parallel with > >> [Build]'? AFAIK the checks need the build results to run themselves. > >> > >> On Wed, 3 Mar 2021 at 18:48, Max Timonin <timonin.ma...@gmail.com> wrote: > >> > > >> > Discussed with Petr privately. Proposal is: > >> > > >> > 1. The [Build] job runs without any checks. > >> > 2. There will be a new job [Sanity Checks], that runs all checks - > >> > checkstyle, licenses, javadoc, check-suites. > >> > 3. [Sanity Checks] runs in parallel with [Build]. > >> > 4. All TC jobs with tests depend on a result of the [Sanity Checks] > >> job. If > >> > the check job fails then a test job won't be started. > >> > 5. Users can disable the [Sanity Checks] job with a selector on the > >> > Parameters tab of custom TC build. > >> > > >> > If no one has objections I will create a JIRA ticket for that. > >> > > >> > > >> > On Wed, Mar 3, 2021 at 5:11 PM Max Timonin <timonin.ma...@gmail.com> > >> wrote: > >> > > >> > > 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 > >> > >> >>>>>>>>>>>>>>> > >> > >> >>>>>>>>>>>>>> > >> > >> >>>>>>>>>>>>> > >> > >> >>>>>>>>>>>> > >> > >> >>>>>>>>>>> > >> > >> >>>>>>>>>> > >> > >> >>>>>>>>> > >> > >> >>>>>>>> > >> > >> >>>>>>> > >> > >> >>>>>> > >> > >> >>>> > >> > >> >>> > >> > >> >>> > >> > >> >> > >> > >> > >> > >> > >> > >
