----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71034/#review216621 -----------------------------------------------------------
Fix it, then Ship it! src/docker/executor.cpp Lines 985-987 (patched) <https://reviews.apache.org/r/71034/#comment303835> Nit: Add `// Need to disambiguate overloaded function.` - Joseph Wu On July 10, 2019, 1:17 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71034/ > ----------------------------------------------------------- > > (Updated July 10, 2019, 1:17 p.m.) > > > Review request for mesos, Benno Evers, Benjamin Mahler, and Joseph Wu. > > > Bugs: MESOS-9853 > https://issues.apache.org/jira/browse/MESOS-9853 > > > Repository: mesos > > > Description > ------- > > This adds a new `killTask()` overload to the Docker executor > and updates the Mesos executor driver to call into that > overload when the loaded executor is the Docker executor. > > This allows the executor driver to pass the kill policy > override, when present, into the Docker executor. > > > Diffs > ----- > > src/docker/executor.hpp f21e84c71f646e84404c65fc2ded64bcaff482ef > src/docker/executor.cpp f638e4b65155bcca1be36424b7061ea26a3d6ca3 > src/exec/exec.cpp c0fa3b61667da96bc4395bae9956c54446268122 > > > Diff: https://reviews.apache.org/r/71034/diff/2/ > > > Testing > ------- > > Details at the end of this chain. > > > Thanks, > > Greg Mann > >
