Yes, we can merge 'binary basic' into 'basic' with adding following tests
into 'basic':
o.a.i.internal.processors.cache.IgniteMarshallerCacheClassNameConflictTest
-testCachePutGetClassesWithNameConflict()

o.a.i.internal.processors.cache.IgniteMarshallerCacheClientRequestsMappingOnMissTest
-testAllNodesDieOnRequest()
-testNoNodesDieOnRequest()
-testOneNodeDiesOnRequest()
-testRequestedMappingIsStoredInFS()
-testTwoNodesDieOnRequest()

it will save 37 minutes of agent time more. I'll prepare PR

Best Regards,
Dmitry Pavlov


вт, 23 мая 2017 г. в 10:43, Vladimir Ozerov <voze...@gridgain.com>:

> Pavel,
>
> Yes, these two:
>
> http://ci.ignite.apache.org/admin/editBuildParams.html?id=buildType:Ignite20Tests_IgniteBasic
>
> http://ci.ignite.apache.org/admin/editBuildParams.html?id=buildType:Ignite20Tests_IgniteBinaryBasic
>
> They both execute org.apache.ignite.testsuites.IgniteBasicTestSuite. Looks
> like we should merge them accurately.
>
> On Tue, May 23, 2017 at 10:27 AM, Dmitry Pavlov <dpavlov....@gmail.com>
> wrote:
>
> > Hi, Vladimir,
> >
> > There is a chance that in
> > http://ci.ignite.apache.org/project.html?projectId=Ignite20Tests&tab=
> > projectOverview
> > some
> > configs are for binary + simple oblect names (no package name) tests.
> >
> > But if there is some suspicious link that is candidate to removal, please
> > feel free to contact me.
> >
> > Best Regards,
> > Dmitry Pavlov
> >
> > вт, 23 мая 2017 г. в 10:24, Vladimir Ozerov <voze...@gridgain.com>:
> >
> > > Fantastic news, thank you guys! Though, I still see "Basic" and "Binary
> > > Object Basic" suites. Should we drop one of them as well?
> > >
> > > On Tue, May 23, 2017 at 10:20 AM, Alexey Goncharuk <
> > > alexey.goncha...@gmail.com> wrote:
> > >
> > > > Igniters,
> > > >
> > > > Since we removed OptimizedMarshaller in Ignite 2.0 from the
> PublicAPI,
> > we
> > > > had a chance to remove several unnecessary test suites from the build
> > > plan
> > > > from Ignite 2.0. I pushed the changes for IGNITE-4947 ticket. From
> this
> > > > moment you should run tests from Ignite 2.0 project, which is 14 test
> > > > suites shorter than the previous plan.
> > > >
> > > > Kudos to Alexander Paschenko and Dmitriy Pavlov!
> > > >
> > > > --AG
> > > >
> > >
> >
>

Reply via email to