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/01/10 00:45:59 UTC

Review Request 16771: Updated slave to use new memory struct.

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

Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.


Repository: mesos-git


Description
-------

>From #16769, memory() now returns both total _and_ free system memory. This patch adjusts for the new return type.


Diffs
-----

  src/slave/slave.cpp 396293b 

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


Testing
-------

make check and functional testing.


Thanks,

Niklas Nielsen


Re: Review Request 16771: Updated slave to use new memory struct.

Posted by Ben Mahler <be...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16771/#review31644
-----------------------------------------------------------

Ship it!


Ship It!

- Ben Mahler


On Jan. 9, 2014, 11:45 p.m., Niklas Nielsen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16771/
> -----------------------------------------------------------
> 
> (Updated Jan. 9, 2014, 11:45 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> From #16769, memory() now returns both total _and_ free system memory. This patch adjusts for the new return type.
> 
> 
> Diffs
> -----
> 
>   src/slave/slave.cpp 396293b 
> 
> Diff: https://reviews.apache.org/r/16771/diff/
> 
> 
> Testing
> -------
> 
> make check and functional testing.
> 
> 
> Thanks,
> 
> Niklas Nielsen
> 
>


Re: Review Request 16771: Updated slave to use new memory struct.

Posted by Benjamin Hindman <be...@berkeley.edu>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16771/#review31490
-----------------------------------------------------------

Ship it!


Ship It!

- Benjamin Hindman


On Jan. 9, 2014, 11:45 p.m., Niklas Nielsen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16771/
> -----------------------------------------------------------
> 
> (Updated Jan. 9, 2014, 11:45 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> From #16769, memory() now returns both total _and_ free system memory. This patch adjusts for the new return type.
> 
> 
> Diffs
> -----
> 
>   src/slave/slave.cpp 396293b 
> 
> Diff: https://reviews.apache.org/r/16771/diff/
> 
> 
> Testing
> -------
> 
> make check and functional testing.
> 
> 
> Thanks,
> 
> Niklas Nielsen
> 
>