mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vinod Kone" <vinodk...@gmail.com>
Subject Re: Review Request: Slave Restart (Part Tres): StatusUpdatesManager integration into the slave
Date Wed, 12 Dec 2012 23:13:42 GMT

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

(Updated Dec. 12, 2012, 11:13 p.m.)


Review request for mesos, Benjamin Hindman and Ben Mahler.


Changes
-------

re-based off trunk.

the only change is that _forwardUpdate now takes Future<Nothing> instead of Future<bool>,
due to the corresponding change in the status manager api.


Description
-------

Integrated SUM into slave.


Diffs (updated)
-----

  include/mesos/mesos.proto 38235157d45bdccb676e5c3241c21b585a6f8801 
  src/Makefile.am c94736df660a25b58dc47c07d9c56c3c26152a66 
  src/slave/gc.cpp fe22f9b557215514e3d432d36af9dc0c377c437b 
  src/slave/paths.hpp fbf3fd84fb8f2590311b18d2afec2d2e0d30ef0a 
  src/slave/slave.hpp e9f7b659ca2860501840b3d01e69915ebd162039 
  src/slave/slave.cpp 9755b46f97173d6fcc9ab1fd63e0e4814b3bc018 
  src/slave/status_update_manager.hpp PRE-CREATION 
  src/tests/master_tests.cpp 948ab5dff34eeba1f3ce593a864ddf282c8b19ed 
  src/tests/status_update_manager_tests.cpp PRE-CREATION 

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


Testing
-------

make check


Thanks,

Vinod Kone


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message