mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Niklas Nielsen" <...@qni.dk>
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:
https://reviews.apache.org/r/18403/
-----------------------------------------------------------

(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
    https://issues.apache.org/jira/browse/MESOS-922


Repository: mesos-git


Description
-------

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 

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


Testing
-------

make check


Thanks,

Niklas Nielsen


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