mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joseph Wu <>
Subject Review Request 63781: Updated Docker path for Containerizer::launch interface change.
Date Tue, 14 Nov 2017 01:38:16 GMT

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

Review request for mesos, Gilbert Song and Jie Yu.

Bugs: MESOS-7305

Repository: mesos


The change in the return type for Containerizer::launch has less of an
impact on the Docker containerizer, as it (currently) will not return
the "new" enum 'ALREADY_LAUNCHED'.

Note that in changing Containerizer::launch, a private helper of the
Docker containerizer `reapExecutor`s return value was changed.
The return value was originally `Future<bool>` because `reapExecutor`
is the final continuation in the launch path so it needed to match
the return value of Containerizer::launch.  However, `reapExecutor`
never returns `false` (only `true` or a Failure).


  src/slave/containerizer/docker.hpp b602a5698cae12686f51c4b9370a06042cda6270 
  src/slave/containerizer/docker.cpp 292eecbca246edf068ec8c262aff4f3ce9cd8c67 



Requires the next review in the chain (test changes).


Joseph Wu

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