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

Ship it!



src/master/master.cpp
<https://reviews.apache.org/r/23661/#comment84443>

    Why not just pass the framework pid instead of Framework*?


- Vinod Kone


On July 18, 2014, 1:51 a.m., Ben Mahler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23661/
> -----------------------------------------------------------
> 
> (Updated July 18, 2014, 1:51 a.m.)
> 
> 
> Review request for mesos, Benjamin Hindman and Vinod Kone.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> All status updates in the master are now sent via Master::forward() which 
> means they are all logged in the same format.
> 
> This should make debugging easier and it simplifies the Master code.
> 
> 
> Diffs
> -----
> 
>   src/common/protobuf_utils.hpp 12ff00a2e1262d785dbd6ec4d54e50d88d611e20 
>   src/master/master.hpp fa46a67fba0fc0a019cdb37770994eee0099d260 
>   src/master/master.cpp fb2fd5a0288296370a034d57083cf22992306cea 
> 
> Diff: https://reviews.apache.org/r/23661/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Ben Mahler
> 
>

Reply via email to