You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Kirk Lund <ki...@gmail.com> on 2016/09/02 00:18:53 UTC
Review Request 51590: GEODE-1838: remove unused test classes
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51590/
-----------------------------------------------------------
Review request for geode, Jason Huynh and nabarun nag.
Bugs: GEODE-1838
https://issues.apache.org/jira/browse/GEODE-1838
Repository: geode
Description
-------
GEODE-1838: remove unused test classes
For GEODE-1832, the changes shown in https://reviews.apache.org/r/51589/ removes the only dependency on the classes in the cacheRunner package.
Diffs
-----
geode-core/src/test/java/cacheRunner/Portfolio.java 5dd2b7e
geode-core/src/test/java/cacheRunner/Position.java 4e7e6e1
Diff: https://reviews.apache.org/r/51590/diff/
Testing
-------
clean build, precheckin
Thanks,
Kirk Lund
Re: Review Request 51590: GEODE-1838: remove unused test classes
Posted by Jason Huynh <jh...@pivotal.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51590/#review147621
-----------------------------------------------------------
Ship it!
Ship It!
- Jason Huynh
On Sept. 2, 2016, 12:18 a.m., Kirk Lund wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51590/
> -----------------------------------------------------------
>
> (Updated Sept. 2, 2016, 12:18 a.m.)
>
>
> Review request for geode, Jason Huynh and nabarun nag.
>
>
> Bugs: GEODE-1838
> https://issues.apache.org/jira/browse/GEODE-1838
>
>
> Repository: geode
>
>
> Description
> -------
>
> GEODE-1838: remove unused test classes
>
> For GEODE-1832, the changes shown in https://reviews.apache.org/r/51589/ removes the only dependency on the classes in the cacheRunner package.
>
>
> Diffs
> -----
>
> geode-core/src/test/java/cacheRunner/Portfolio.java 5dd2b7e
> geode-core/src/test/java/cacheRunner/Position.java 4e7e6e1
>
> Diff: https://reviews.apache.org/r/51590/diff/
>
>
> Testing
> -------
>
> clean build, precheckin
>
>
> Thanks,
>
> Kirk Lund
>
>