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