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

Posted to dev@mesos.apache.org by Vinod Kone <vi...@gmail.com> on 2013/03/13 07:15:52 UTC

Re: Review Request: Slave Restart (Part 2): Status Update Manager.

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

(Updated March 13, 2013, 6:15 a.m.)


Review request for mesos, Benjamin Hindman and Ben Mahler.


Changes
-------

s/part 1/part 2/


Summary (updated)
-----------------

Slave Restart (Part 2): Status Update Manager.


Description
-------

added shutdownFramework


Minor fixes for open


Ben's comments


Refactoring SUM


Bens' comments


Status Update Manager

Rebased off latest trunk

Conflicts:
	src/Makefile.am
	src/common/protobuf_utils.hpp
	src/common/utils.hpp
	src/slave/slave.cpp
	src/tests/protobuf_io_tests.cpp
	src/tests/utils_tests.cpp


Diffs
-----

  src/Makefile.am f7242a7fb55de3d1df4930126345274124331d57 
  src/common/protobuf_utils.hpp 69baab78c8db2d2d33ffbbe7f5e5fc80d65b0e1a 
  src/common/type_utils.hpp 66c2dc17655123641433c078931acc9e769d5077 
  src/local/local.cpp 3402029e07341212717e346a97e7cd30fa52ed51 
  src/messages/messages.proto 815fcbbcb4a8643f50950a294cedf7281b2a187f 
  src/slave/status_update_manager.hpp PRE-CREATION 
  src/slave/status_update_manager.cpp PRE-CREATION 
  third_party/libprocess/include/process/timeout.hpp cac996070359a3e7ecdd8077af83c8c4cf9735fd 

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


Testing
-------

make check.


Thanks,

Vinod Kone