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