IGNITE-603

[Александр Меньшиков]

Hello, everyone.

I think the commented-out method "processSize2Test" can be removed. There
aren't class "GridDeploymentMode" which used in arguments.

Do you agree with me?

[dmagda]

Hi Alexander,

Looks like the test is commented for a while. GridDeploymentMode has already been renamed to DeploymentMode.

Yakov, is this test still relevant for the compute grid?

—
Denis

> On Feb 13, 2017, at 5:45 AM, Александр Меньшиков <[hidden email]> wrote:
>
> Hello, everyone.
>
> I think the commented-out method "processSize2Test" can be removed. There
> aren't class "GridDeploymentMode" which used in arguments.
>
> Do you agree with me?

[yzhdanov]

Denis, I can't say for sure, but the fact test was commented out for a long
time is suspicious.

Alexander, can you please uncomment, fix all the compilation errors and
update the ticket with test run results. Make sure you have
modules/ext-data projects built and all paths mentioned
in modules/core/src/test/config/tests.properties are valid (you will need
to set IGNITE_HOME env var most probably)

--Yakov

2017-02-13 23:10 GMT+03:00 Denis Magda <[hidden email]>:

> Hi Alexander,
>
> Looks like the test is commented for a while. GridDeploymentMode has
> already been renamed to DeploymentMode.
>
> Yakov, is this test still relevant for the compute grid?
>
> —
> Denis
>
> > On Feb 13, 2017, at 5:45 AM, Александр Меньшиков <[hidden email]>
> wrote:
> >
> > Hello, everyone.
> >
> > I think the commented-out method "processSize2Test" can be removed. There
> > aren't class "GridDeploymentMode" which used in arguments.
> >
> > Do you agree with me?
>
>

[Александр Меньшиков]

Yakov, Okay. You can see result in JIRA
https://issues.apache.org/jira/browse/IGNITE-603

2017-02-28 19:31 GMT+03:00 Yakov Zhdanov <[hidden email]>:

> Denis, I can't say for sure, but the fact test was commented out for a long
> time is suspicious.
>
> Alexander, can you please uncomment, fix all the compilation errors and
> update the ticket with test run results. Make sure you have
> modules/ext-data projects built and all paths mentioned
> in modules/core/src/test/config/tests.properties are valid (you will need
> to set IGNITE_HOME env var most probably)
>
> --Yakov
>
> 2017-02-13 23:10 GMT+03:00 Denis Magda <[hidden email]>:
>
> > Hi Alexander,
> >
> > Looks like the test is commented for a while. GridDeploymentMode has
> > already been renamed to DeploymentMode.
> >
> > Yakov, is this test still relevant for the compute grid?
> >
> > —
> > Denis
> >
> > > On Feb 13, 2017, at 5:45 AM, Александр Меньшиков <[hidden email]
> >
> > wrote:
> > >
> > > Hello, everyone.
> > >
> > > I think the commented-out method "processSize2Test" can be removed.
> There
> > > aren't class "GridDeploymentMode" which used in arguments.
> > >
> > > Do you agree with me?
> >
> >
>

[Yakov Zhdanov-2]

Thanks, Alex! I will take a look in a couple of days

Thanks!
--
Yakov Zhdanov, Director R&D
*GridGain Systems*
www.gridgain.com

2017-03-02 19:07 GMT+03:00 Александр Меньшиков <[hidden email]>:

> Yakov, Okay. You can see result in JIRA https://issues.apache.org/
> jira/browse/IGNITE-603
>
> 2017-02-28 19:31 GMT+03:00 Yakov Zhdanov <[hidden email]>:
>
>> Denis, I can't say for sure, but the fact test was commented out for a
>> long
>> time is suspicious.
>>
>> Alexander, can you please uncomment, fix all the compilation errors and
>> update the ticket with test run results. Make sure you have
>> modules/ext-data projects built and all paths mentioned
>> in modules/core/src/test/config/tests.properties are valid (you will need
>> to set IGNITE_HOME env var most probably)
>>
>> --Yakov
>>
>> 2017-02-13 23:10 GMT+03:00 Denis Magda <[hidden email]>:
>>
>> > Hi Alexander,
>> >
>> > Looks like the test is commented for a while. GridDeploymentMode has
>> > already been renamed to DeploymentMode.
>> >
>> > Yakov, is this test still relevant for the compute grid?
>> >
>> > —
>> > Denis
>> >
>> > > On Feb 13, 2017, at 5:45 AM, Александр Меньшиков <
>> [hidden email]>
>> > wrote:
>> > >
>> > > Hello, everyone.
>> > >
>> > > I think the commented-out method "processSize2Test" can be removed.
>> There
>> > > aren't class "GridDeploymentMode" which used in arguments.
>> > >
>> > > Do you agree with me?
>> >
>> >
>>
>
>