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

Ship it!



src/slave/constants.cpp
<https://reviews.apache.org/r/8077/#comment38227>

    plumb it through flags.



src/slave/slave.cpp
<https://reviews.apache.org/r/8077/#comment38228>

    kill comment



src/slave/slave.cpp
<https://reviews.apache.org/r/8077/#comment38229>

    kill comment



src/slave/slave.cpp
<https://reviews.apache.org/r/8077/#comment38230>

    s/WARNING/INFO
    
    rephrase



src/slave/slave.cpp
<https://reviews.apache.org/r/8077/#comment38231>

    flags.


- Vinod Kone


On March 16, 2013, 1:49 a.m., Vinod Kone wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8077/
> -----------------------------------------------------------
> 
> (Updated March 16, 2013, 1:49 a.m.)
> 
> 
> Review request for mesos, Benjamin Hindman and Ben Mahler.
> 
> 
> Description
> -------
> 
> This paves the way for a slave to shutdown executors that are not responsive 
> during launch (e.g. HDFS hang during fetching executor, executor process hung 
> after fork).
> 
> 
> This addresses bug MESOS-310.
>     https://issues.apache.org/jira/browse/MESOS-310
> 
> 
> Diffs
> -----
> 
>   src/master/hierarchical_allocator_process.hpp 
> c1d6f54fb0c93d255f95ee85e1305022db1ce3cc 
>   src/master/master.cpp 87bbfac692c3f37f4ef1112b1a44e2afa01296d8 
>   src/slave/cgroups_isolator.hpp 049d50e306d936708d2bde72b1f16f8accaf9a13 
>   src/slave/cgroups_isolator.cpp c06baecac5a89886673a36d3dad934e06a577875 
>   src/slave/constants.hpp 02c45398cbc2686de22750c38f69761e0fccb24c 
>   src/slave/constants.cpp 8ef3642d1624db71f40a07be86373f44afa93f21 
>   src/slave/process_isolator.cpp e719a0711157f37cf87b92127fd4f50287b1cf21 
>   src/slave/slave.hpp dadd06850565d07b46cea2f581661fb57a121118 
>   src/slave/slave.cpp 912499e4cb34afa9bcc1556fd9536581001933ca 
>   src/tests/master_tests.cpp 9d13ebe2f1d3cbc1d39fd8bdd5060a15e2767eb0 
>   third_party/libprocess/include/process/delay.hpp 
> cb2fa9ab372e62befec01a95b3ca401567a1a626 
> 
> Diff: https://reviews.apache.org/r/8077/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Vinod Kone
> 
>

Reply via email to