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




src/exec/exec.cpp (line 693)
<https://reviews.apache.org/r/43760/#comment182459>

    I pulled down these patches to tweak them as I wanted to add the 
configurable shutdown grace period. If we kept in mind that the shutdown grace 
period should be configurable, this would be 
'MESOS_DEFAULT_EXECUTOR_SHUTDOWN_GRACE_PERIOD'. It would be just a default 
specified by the agent, that can be overridden by ExecutorInfo.
    
    In the new API (see src/executor/executor.cpp), it looks like we still need 
to send this default through an environment variable because we have 
self-initiated shutdowns. These self-initiated shutdowns won't have the 
'Shutdown' Event's grace period so the executor needs to know the agent's 
default outside of the 'Shutdown' Event.
    
    Hope this makes some sense about how I'm thinking about tweaking this. I'll 
send out the tweaked patches and we can discuss!



src/exec/exec.cpp (lines 697 - 698)
<https://reviews.apache.org/r/43760/#comment182457>

    I see "Cannot parse" was copied, but the language we use to consistently 
print failures is:
    
    "Failed to <do something>: <reason>"


- Ben Mahler


On Feb. 19, 2016, 1:12 p.m., Alexander Rukletsov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43760/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2016, 1:12 p.m.)
> 
> 
> Review request for mesos, Anand Mazumdar and Ben Mahler.
> 
> 
> Bugs: MESOS-1571
>     https://issues.apache.org/jira/browse/MESOS-1571
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Executor shutdown grace period, which configured on the agent, is
> propagated to executors via the `MESOS_EXECUTOR_SHUTDOWN_GRACE_PERIOD`
> environment variable. The executor library uses this timeout to delay
> the hard shutdown of the related executor.
> 
> 
> Diffs
> -----
> 
>   src/exec/exec.cpp 83dbee9dd2d9a4e7ebf395c8070bc7f9f8412ef1 
>   src/slave/containerizer/containerizer.cpp 
> 59904684cdeb17ef2b42092a3558802c42bfb6bd 
> 
> Diff: https://reviews.apache.org/r/43760/diff/
> 
> 
> Testing
> -------
> 
> The complete chain was tested. See https://reviews.apache.org/r/43764/ .
> 
> 
> Thanks,
> 
> Alexander Rukletsov
> 
>

Reply via email to