You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Jie Yu <yu...@gmail.com> on 2017/12/22 19:46:36 UTC

Review Request 64811: Cleaned up ContainerInfo generation logic in the slave.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64811/
-----------------------------------------------------------

Review request for mesos, Chun-Hung Hsiao and Gilbert Song.


Bugs: MESOS-7007
    https://issues.apache.org/jira/browse/MESOS-7007


Repository: mesos


Description
-------

After this patch, the invariant becomes that for command tasks, we
always set the ContainerInfo of the generated ExecutorInfo to be the
same as that in TaskInfo (if exists). This simplifies the logic when we
actually generate ContainerInfo for containerizer during launching
phase.

This also made the logic introduced in the following patch more readable
and maintainable: https://reviews.apache.org/r/63598


Diffs
-----

  src/slave/slave.cpp 47d5632115bbfd2b729dc2388c01733f907cf938 


Diff: https://reviews.apache.org/r/64811/diff/1/


Testing
-------

sudo make check


Thanks,

Jie Yu