Igniters,

I've created new PR https://github.com/apache/ignite/pull/1986 without
merge conflicts against master. Please review these changes instead of
PR1956.

Thank you

Best Regards,
Dmitry Pavlov

пн, 22 мая 2017 г. в 13:21, Alexey Goncharuk <alexey.goncha...@gmail.com>:

> Thanks, Dmitriy!
>
> If there are no other objections, I will merge the changes shortly.
>
> 2017-05-22 13:10 GMT+03:00 Dmitry Pavlov <dpavlov....@gmail.com>:
>
> > Hi, Igniters!
> >
> > Could you please review and merge changes from
> > https://github.com/apache/ignite/pull/1956 into master.
> >
> > These changes makes running tests with similar results in TC Run configs
> > "Ignite 2.0 Tests" as it is in "Ignite Tests".
> >
> > Best regards,
> > Dmitry Pavlov
> >
> > ср, 17 мая 2017 г. в 20:52, Alexey Goncharuk <alexey.goncha...@gmail.com
> >:
> >
> > > Great news! Looking forward to getting rid of the unnecessary TC
> > > configurations.
> > >
> > > Once this is done, I think we should also work out if we can use build
> > > once, run tests approach for RunAll configuration. Does anybody have a
> > clue
> > > if this is possible?
> > >
> > > 2017-05-17 20:46 GMT+03:00 Dmitry Pavlov <dpavlov....@gmail.com>:
> > >
> > > > Hi, Igniters!
> > > >
> > > > Alex (al.psc) has created new Teamcity run configurations optimized
> for
> > > > running 2.0 version and later. This project and its run
> configurations
> > > > avoid duplicate tests running for binary and default marshallers
> > because
> > > it
> > > > is now the same for 2.0+.
> > > >
> > > > Usage of this run configuration saves us at least 12 machine-hours
> for
> > > each
> > > > test run and in future will allow us to get test results faster.
> > > >
> > > > Do you have concerns or remarks about using new run configurations
> for
> > > all
> > > > 2.0+ pull requests?
> > > >
> > > > We are now doing final checks of compatibility of these run
> > > configurations
> > > > with code. Any ideas about what is to be migrated are appreciated.
> > > >
> > > > Best Regards,
> > > > Dmitry Pavlov
> > > >
> > >
> >
>

Reply via email to