Re: Review Request 51148: Moved MockSlave into a separate header/implementation file.

2016-09-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51148/#review148727 --- Patch looks great! Reviews applied: [51149, 51150, 51151, 51808,

Re: Review Request 51148: Moved MockSlave into a separate header/implementation file.

2016-09-13 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51148/ --- (Updated Sept. 13, 2016, 10:21 a.m.) Review request for mesos and Michael Park.

Re: Review Request 51148: Moved MockSlave into a separate header/implementation file.

2016-09-12 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51148/ --- (Updated Sept. 12, 2016, 2:36 p.m.) Review request for mesos and Michael Park.

Re: Review Request 51148: Moved MockSlave into a separate header/implementation file.

2016-09-12 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51148/#review148494 --- Ship it! I think we need to add the `mock_slave.hpp` into `Make

Review Request 51148: Moved MockSlave into a separate header/implementation file.

2016-08-16 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51148/ --- Review request for mesos and Michael Park. Repository: mesos Description