mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Toenshoff" <toensh...@me.com>
Subject Re: Review Request 40107: Removed unused checks in command executor.
Date Tue, 10 Nov 2015 13:02:11 GMT

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


This is supposed to be the fix (or workaround) for MESOS-3851, right? Let's please tag it
in the "bugs" section. However, just as Vinod commented in that ticket, it seems the implications
of the problem might really be bigger and not only localized to tbe command executor.

- Till Toenshoff


On Nov. 9, 2015, 9:37 p.m., Timothy Chen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40107/
> -----------------------------------------------------------
> 
> (Updated Nov. 9, 2015, 9:37 p.m.)
> 
> 
> Review request for mesos, Anand Mazumdar and Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Removed unused checks in command executor.
> 
> 
> Diffs
> -----
> 
>   src/launcher/executor.cpp 0dad75d7d730ef6cdc4b427a358625433cfee510 
> 
> Diff: https://reviews.apache.org/r/40107/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Timothy Chen
> 
>


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