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