incubator-mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vinod Kone" <>
Subject Re: Review Request: Slave Restart (Part Uno): Factoring out Slave State
Date Wed, 12 Sep 2012 22:23:09 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Sept. 12, 2012, 10:23 p.m.)

Review request for mesos, Benjamin Hindman and Ben Mahler.


benm's comments.

also, diff'ed against apache/trunk, after  a huuuuuge rebase. please check carefully :)


And so it begins...

I am discarding the old review associated with this (previously called path:,
in favor of a fresh start.

slave/state.{hpp,cpp} contains all the functions that deal with persistent state of the slave.

This addresses bug MESOS-110.

Diffs (updated)

  src/ b73a024 
  src/common/protobuf_utils.hpp PRE-CREATION 
  src/slave/slave.hpp 7fbae8a 
  src/slave/slave.cpp 4ea1db1 
  src/slave/state.hpp PRE-CREATION 
  src/slave/state.cpp PRE-CREATION 
  src/tests/slave_state_tests.cpp PRE-CREATION 
  third_party/libprocess/include/stout/format.hpp c746ad3 
  third_party/libprocess/include/stout/os.hpp d08b0cb 



make check


Vinod Kone

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