-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29600/#review67740
-----------------------------------------------------------



include/mesos/mesos.proto
<https://reviews.apache.org/r/29600/#comment111827>

    Can we explain in the comments why a internal struct is then exposed in 
mesos.proto?


- Timothy Chen


On Jan. 10, 2015, 7:26 a.m., Kapil Arya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29600/
> -----------------------------------------------------------
> 
> (Updated Jan. 10, 2015, 7:26 a.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
> -------
> 
> [2/10] Build Mesos Modules outside Mesos source tree.
> 
> This prepares us to expose src/slave/containerizer/isolator.hpp as 
> <mesos/slave/isolator.hpp> and src/slave/state.hpp as <mesos/slave/state.hpp> 
> for inclusion for Isolator modules.
> 
> 
> Diffs
> -----
> 
>   include/mesos/mesos.proto 6c9f5d351bf7befc0a4956d0176018c80d96418d 
>   src/common/http.hpp afce7fea334c7bfa57efc48c413bf906a2ebde32 
>   src/common/type_utils.hpp 2d22db85f87ec5c1134208aae5b2105d2d267000 
>   src/common/type_utils.cpp fcc9eb0e27a9cbf53f7923f0c051a6578527ec37 
>   src/messages/messages.proto 5bd075e9bfe17be334294b5e4c573eb0233030be 
>   src/slave/state.hpp 592182b21ae2700c627f180db38ca5dc1b3c9e8a 
> 
> Diff: https://reviews.apache.org/r/29600/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Kapil Arya
> 
>

Reply via email to