Re: Review Request 54686: Moved the `IOSwitchboardServer::isRequired` to `IOSwitchBoard`.

2016-12-12 Thread Vinod Kone
> On Dec. 13, 2016, 12:54 a.m., Kevin Klues wrote: > > src/slave/containerizer/mesos/io/switchboard.hpp, lines 81-99 > > > > > > Is there a reason not to put the implementation of this in the .cpp > > file? Seems

Re: Review Request 54686: Moved the `IOSwitchboardServer::isRequired` to `IOSwitchBoard`.

2016-12-12 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54686/#review158941 --- Fix it, then Ship it!

Review Request 54686: Moved the `IOSwitchboardServer::isRequired` to `IOSwitchBoard`.

2016-12-12 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54686/ --- Review request for mesos, Alex Clemmer and Kevin Klues. Repository: mesos