incubator-mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vinod Kone" <>
Subject Re: Review Request: Added registration timeout for executors.
Date Mon, 04 Mar 2013 21:51:12 GMT

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

(Updated March 4, 2013, 9:51 p.m.)

Review request for mesos, Benjamin Hindman and Ben Mahler.


Rebased off of "Properly cleanup cgroups"

Summary (updated)

Added registration timeout for executors.


This paves the way for a slave to shutdown executors that are not responsive during launch
(e.g. HDFS hang during fetching executor).

This fix is not enough for cgroups isolation module (enough for the process based isolation
module) because it currently blocks while launching an executor.

This addresses bug MESOS-310.

Diffs (updated)

  src/master/hierarchical_allocator_process.hpp 33e059c10bb87db88b1fb9a100b7504bc89ac896 
  src/master/master.cpp 814a6e1af2c7caf77452748dfc9e6935d8386d8f 
  src/slave/cgroups_isolation_module.hpp a04fc46b15d2741886f5847cadbdc9bed351c588 
  src/slave/constants.hpp dc5b9877e2608f6806f13b6ef4d133b6fddaece4 
  src/slave/constants.cpp ecb96fd42afdb74aca394411c6ff60276f509cfd 
  src/slave/process_based_isolation_module.hpp 6a4e6aef9ab4b13cce9341f6f35c6b24e6f8fde7 
  src/slave/process_based_isolation_module.cpp ff98d105af675dfc66070feaa43b42c1aa438fd8 
  src/slave/reaper.cpp c0ee4b4c07fd792bcb39455b666808b712eb32c2 
  src/slave/slave.hpp 7648c33230c1900eda7529045c5df9ccab105d47 
  src/slave/slave.cpp c13d268c5e0e107902f64e30304a18128927a571 
  src/tests/master_tests.cpp 104610cef3fdaaea75271657c42c4fd2b6b61618 
  third_party/libprocess/include/process/delay.hpp cb2fa9ab372e62befec01a95b3ca401567a1a626



make check


Vinod Kone

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