mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Niklas Nielsen" <>
Subject Re: Review Request 18403: Added support for launching tasks by TaskInfo.
Date Mon, 17 Mar 2014 17:37:39 GMT

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

(Updated March 17, 2014, 10:37 a.m.)

Review request for mesos, Adam B, Benjamin Hindman, Ben Mahler, Ian Downes, Till Toenshoff,
and Vinod Kone.

Bugs: MESOS-922

Repository: mesos-git


This patch delegates the choice of executor to the containerizer by removing executorInfo
dependencies up until Containerizer::launch().
Containerizer::launch() now returns a future to the executor info that is being run and the
slave creates the corresponding executor structure when launch completes.
This means message handling from the running executor to the slave in the interim where the
executor structure has not created, need to be enqueued until executor is ready. So far, registerExecutor()
and reregisterExecutor() has been split into two continuations to deal with this issue.

Diffs (updated)

  src/slave/containerizer/containerizer.hpp d9ae326 
  src/slave/containerizer/mesos_containerizer.hpp ee1fd30 
  src/slave/containerizer/mesos_containerizer.cpp c819c97 
  src/slave/http.cpp 594032d 
  src/slave/slave.hpp 01b80df 
  src/slave/slave.cpp d8d3e0f 
  src/tests/containerizer.hpp 5686398 
  src/tests/containerizer.cpp bfb9341 



make check


Niklas Nielsen

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