----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29600/#review66940 -----------------------------------------------------------
include/mesos/mesos.proto <https://reviews.apache.org/r/29600/#comment110672> I think this can be confusing pushing this into the top level, as it becomes unclear what this is used for especially with a very generic name with Task. What do you think about wrapping these with a internal namespace, so at least it's clear it's not intended as a public API. - Timothy Chen On Jan. 6, 2015, 10:39 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29600/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2015, 10:39 p.m.) > > > Review request for mesos, Niklas Nielsen, Till Toenshoff, and Timothy Chen. > > > Bugs: MESOS-2096 > https://issues.apache.org/jira/browse/MESOS-2096 > > > Repository: mesos-git > > > Description > ------- > > [3/9] Build Mesos Modules outside Mesos source tree. > > This prepares us to expose src/slave/containerizer/isolator.hpp as > <mesos/isolator.hpp> to be used for building mesos modules outside mesos > source tree. > > > Diffs > ----- > > include/mesos/mesos.hpp 371d14a6cf802d1099be84f217a1af6554cc4eee > include/mesos/mesos.proto 540071db64961466eb75c779b3ea6863f4594437 > src/common/http.hpp afce7fea334c7bfa57efc48c413bf906a2ebde32 > src/common/type_utils.hpp 2d22db85f87ec5c1134208aae5b2105d2d267000 > src/common/type_utils.cpp fcc9eb0e27a9cbf53f7923f0c051a6578527ec37 > src/messages/messages.proto 16043dd934cbfc1159dad7adf366dfbfcc5a077b > src/slave/state.hpp 592182b21ae2700c627f180db38ca5dc1b3c9e8a > > Diff: https://reviews.apache.org/r/29600/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Kapil Arya > >
