ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Max Timonin <timonin.ma...@gmail.com>
Subject Re: [DISCUSS] Missed (non-suited) tests
Date Mon, 08 Feb 2021 07:21:42 GMT
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.kasnacheev@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.kasnacheev@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.sin@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.maxim@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.kasnacheev@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.maxim@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 <
> ivandasch@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.kasnacheev@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.maxim@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.kasnacheev@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.maxim@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.maxim@gmail.com
> >> > > > > >
> >> > > > > > > > wrote:
> >> > > > > > > >
> >> > > > > > > > > Sure, I'll do that.
> >> > > > > > > > >
> >> > > > > > > > > On Fri, Nov 27, 2020 at 2:00 PM Ilya Kasnacheev <
> >> > > > > > > > ilya.kasnacheev@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.maxim@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.kasnacheev@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.maxim@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.kasnacheev@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.maxim@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.maxim@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.kasnacheev@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.maxim@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
> >> > > > <
> >> > > > > > > > >> > av@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 <
> >> > > > > > > > >> > > > > > ivandasch@gmail.com>
> >> > > > > > > > >> > > > > > >> > > wrote:
> >> > > > > > > > >> > > > > > >> > >
> >> > > > > > > > >> > > > > > >> > > > I suggests to mark these tests with
> >> > @Ignore
> >> > > > and
> >> > > > > > > file
> >> > > > > > > > >> > tickets
> >> > > > > > > > >> > > > to
> >> > > > > > > > >> > > > > > fix
> >> > > > > > > > >> > > > > > >> > them.
> >> > > > > > > > >> > > > > > >> > > >
> >> > > > > > > > >> > > > > > >> > > > пт, 30 окт. 2020 г. в 16:26, Ivan
> >> > > Daschinsky
> >> > > > <
> >> > > > > > > > >> > > > > ivandasch@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.maxim@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 <
> >> > > > > > > > >> > > > > > av@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
> >> > > > > > > > <
> >> > > > > > > > >> > > > > > >> > > vololo100@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.maxim@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 <
> >> > > > > > > > >> > > > > > >> > > > >> ivandasch@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 <
> >> > > > > > > > >> > > > > > >> > > vololo100@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.maxim@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.stepachev@gmail.com
> >> > :
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > >  GridTcpCommunicationSpiLogTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > nizhikov@apache.org:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >>
> >> > IgniteCacheClientMultiNodeUpdateTopologyLockTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > >  CacheClientsConcurrentStartTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > >  IgniteOutOfMemoryPropagationTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > >  GridCacheP2PUndeploySelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > >  GridCacheRebalancingOrderingTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > >  IgniteMassLoadSandboxTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > >  PageLockTrackerMXBeanImplTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > >  IgniteBinaryMetadataUpdateNodeRestartTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > >  CacheLockCandidatesThreadTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> >  GridMBeanBaselineTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > >  RendezvousAffinityFunctionSimpleBenchmark
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > samvimes@yandex.ru:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > >  IgnitePdsNoSpaceLeftOnDeviceTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > maxmuzaf@gmail.com:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> >  GridCacheOnCopyFlagReplicatedSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > >  GridCacheOnCopyFlagLocalSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >>
> >> > GridCacheReplicatedAtomicReferenceMultiNodeTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >>  GridCacheReplicatedMarshallerTxTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> >  GridCacheReplicatedTxConcurrentGetTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > >  GridCacheOnCopyFlagTxPartitionedSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > >  GridCacheReplicatedTxReadTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >>
> >> > GridCachePartitionedAtomicReferenceMultiNodeTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > >  GridCacheOnCopyFlagAtomicSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > mmuzaf@apache.org:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > >  GridActivateExtensionTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > >  IgniteChangeGlobalStateCacheTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > >  IgniteChangeGlobalStateTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >>  IgniteChangeGlobalStateServiceTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > >  IgniteChangeGlobalStateDataStructureTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> oignatenko@gridgain.com
> >> :
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > >  CacheEntryProcessorCopySelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > >  MemoryLeaksOnRestartNodeTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > >  GridCacheAtomicPreloadSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > >  WalCompactionAfterRestartTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > >  IgniteCacheConcurrentPutGetRemove
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > >  GridIoManagerBenchmark0
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > nsamelchev@gmail.com:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > >
> GridLongRunningInitNewCrdFutureDiagnosticsTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > >  GridCacheMultithreadedFailoverAbstractTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> alexey.goncharuk@gmail.com
> >> > :
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > >  GridCacheBinaryObjectsAtomicOnheapSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > >
> >> > > > > > > > GridCacheBinaryObjectsAtomicNearDisabledOnheapSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >>
> >> > GridCacheBinaryObjectsPartitionedOnheapSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> >
> >> > > > > > > > >> > > > >
> >> > > > > GridCacheBinaryObjectsPartitionedNearDisabledOnheapSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > vladisavj@gmail.com:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > >  IgnitePartitionedLockSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> alexandr.belyak@xored.com:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > >  IgniteStableBaselineCachePutAllFailoverTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > >  IgniteStableBaselineCacheRemoveFailoverTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > ilantukh@gridgain.com
> :
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > >  IgniteCacheAtomicOnheapExpiryPolicyTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > >  IgniteCacheAtomicLocalOnheapExpiryPolicyTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > >  GridCacheReplicatedOnheapFullApiSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > >  GridCacheBinaryObjectsLocalOnheapSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > oignatenko@users.noreply.github.com:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >>  GridCacheTtlManagerEvictionSelfTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > irakov@apache.org:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >>  CommonPoolStarvationCheckpointTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > alievmirza@gmail.com:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> >  RemoveAllDeadlockTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> schugunov@gridgain.com:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >>  FullyConnectedComponentSearcherTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > > sboikov@gridgain.com:
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > >  IgniteDataStructuresNoClassOnServerTest
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> >> > > > > > > > >> > > > > > >> > > > >> > > >> > >
> timonin.maxim@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
> >> > > >
> >> > >
> >> >
> >>
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message