mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nishant Suneja" <>
Subject Re: Review Request 29720: Bug fix: Added an executor container launch timeout timer
Date Sat, 10 Jan 2015 16:21:16 GMT

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

(Updated Jan. 10, 2015, 4:21 p.m.)

Review request for mesos and Timothy Chen.


Changes to the dependent patch.

Bugs: MESOS-999

Repository: mesos-git


A new executor launch timeout timer has been added. This timer gates the time in which a successful
executor container launch should happen. The executor registration timer starts after the
successful container launch.

Diffs (updated)

  src/slave/constants.hpp 761cfafb3b1b342af4d1dbdb2dec39a45dd62794 
  src/slave/constants.cpp 83d9fc1cd385722deb31883c17c824fd0e1de8a6 
  src/slave/flags.hpp f1b8dfbb7391167b67a9498561742aa9ab9089a6 
  src/slave/slave.hpp 70bd8c1fde4ea09fa54c76aa93424a1adb0309f6 
  src/slave/slave.cpp b234f5359a91a293b93f97f035b08ef0a4c9b20d 
  src/tests/slave_tests.cpp a85a0282947278ce6e02e38093f8b88af4cb612a 



make check succeeds.
Added the unit test : SlaveTest:: ExecutorLaunchTimeoutTrigger.
Added the unit test : SlaveTest:: ExecutorNoLaunchTimeoutTrigger.


Nishant Suneja

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