----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44656/#review124339 -----------------------------------------------------------
Fix it, then Ship it! include/mesos/mesos.proto (lines 354 - 356) <https://reviews.apache.org/r/44656/#comment186824> How about removing the first setence in this NOTE and converting it to just a non-NOTE paragraph? You already say this on top of the kill_policy field? include/mesos/mesos.proto (lines 366 - 370) <https://reviews.apache.org/r/44656/#comment186823> How about following similar language from the executor info change? ``` // The grace period specifies how long to wait before forcibly // killing the task. It is recommended to attempt to gracefully // kill the task (and send TASK_KILLING) to indicate that the // graceful kill is in progress. Once the grace period elapses, // if the task has not terminated, a forcible kill should occur. // The task should not assume that it will *always* be alloted // the full grace period. For example, the executor may be // shutdown more quickly by the agent, or failures / forcible // terminations may occur. ``` However, you already have a NOTE above, so I'd rather not call it out again here. - Ben Mahler On March 18, 2016, 5:14 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44656/ > ----------------------------------------------------------- > > (Updated March 18, 2016, 5:14 p.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-4909 > https://issues.apache.org/jira/browse/MESOS-4909 > > > Repository: mesos > > > Description > ------- > > Describes a kill policy for a task. Currently does not express > different policies (e.g. hitting HTTP endpoints). For now, this > controls how long to wait between SIGTERM and SIGKILL. > > > Diffs > ----- > > include/mesos/mesos.proto deb9c0910a27afd67276f54b3f666a878212727b > include/mesos/v1/mesos.proto a981e750c24cfc48177bbc9ca56f0c3ecfae1a1b > > Diff: https://reviews.apache.org/r/44656/diff/ > > > Testing > ------- > > The complete chain was tested. See https://reviews.apache.org/r/44662/. > > > Thanks, > > Alexander Rukletsov > >