On Sept. 8, 2014, 5:42 p.m., Vinod Kone wrote:
src/tests/cluster.hpp, line 597
https://reviews.apache.org/r/25372/diff/1/?file=679951#file679951line597
Since you are cleaning up this code, I don't think this variable is
necessary anymore? Actually I don't know why it was used in
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25372/#review52775
---
Ship it!
Ship It!
- Vinod Kone
On Sept. 9, 2014, 12:43 a.m.,
On Sept. 8, 2014, 5:42 p.m., Vinod Kone wrote:
src/tests/cluster.hpp, line 597
https://reviews.apache.org/r/25372/diff/1/?file=679951#file679951line597
Since you are cleaning up this code, I don't think this variable is
necessary anymore? Actually I don't know why it was used in
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25372/#review52612
---
src/tests/mesos.hpp
On Sept. 8, 2014, 5:42 p.m., Vinod Kone wrote:
src/tests/cluster.hpp, lines 155-164
https://reviews.apache.org/r/25372/diff/1/?file=679951#file679951line155
Similar to Slave do you think these should be Owned?
Good call, cleaned these up as well.
- Ben
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25372/
---
(Updated Sept. 9, 2014, 12:42 a.m.)
Review request for mesos and Vinod Kone.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25372/
---
(Updated Sept. 9, 2014, 12:43 a.m.)
Review request for mesos and Vinod Kone.
On Sept. 8, 2014, 5:42 p.m., Vinod Kone wrote:
src/tests/cluster.hpp, line 597
https://reviews.apache.org/r/25372/diff/1/?file=679951#file679951line597
Since you are cleaning up this code, I don't think this variable is
necessary anymore? Actually I don't know why it was used in
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25372/
---
Review request for mesos and Vinod Kone.
Bugs: MESOS-1715