You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@mesos.apache.org by Niklas Nielsen <ni...@qni.dk> on 2014/09/24 05:54:55 UTC

Review Request 25967: Added task struct in master.

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

Review request for mesos and Ben Mahler.


Repository: mesos-git


Description
-------

In order to hang additional state off tasks (as 'resources_recovered' in https://reviews.apache.org/r/25911/), this patch introduce a task struct to represent tasks within the master.


Diffs
-----

  src/common/http.hpp afce7fe 
  src/master/http.cpp 3f5a01d 
  src/master/master.hpp f5d74ae 
  src/master/master.cpp e5d30e9 

Diff: https://reviews.apache.org/r/25967/diff/


Testing
-------

make check


Thanks,

Niklas Nielsen

Re: Review Request 25967: Added task struct in master.

Posted by Mesos ReviewBot <de...@mesos.apache.org>.

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


Patch looks great!

Reviews applied: [25967]

All tests passed.

- Mesos ReviewBot


On Sept. 24, 2014, 3:54 a.m., Niklas Nielsen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25967/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2014, 3:54 a.m.)
> 
> 
> Review request for mesos and Ben Mahler.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> In order to hang additional state off tasks (as 'resources_recovered' in https://reviews.apache.org/r/25911/), this patch introduce a task struct to represent tasks within the master.
> 
> 
> Diffs
> -----
> 
>   src/common/http.hpp afce7fe 
>   src/master/http.cpp 3f5a01d 
>   src/master/master.hpp f5d74ae 
>   src/master/master.cpp e5d30e9 
> 
> Diff: https://reviews.apache.org/r/25967/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Niklas Nielsen
> 
>