----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71858/#review219692 -----------------------------------------------------------
src/slave/slave.cpp Lines 3201-3204 (original), 3285-3291 (patched) <https://reviews.apache.org/r/71858/#comment307909> So it looks like here, we will only set limits for the executor if they are explicitly set for its tasks. This is different than what we intend to do for task containers, since in that case we will set their limits equal to their requests if `share_cgroups==true` (modulo the value of the agent's `--cgroups_enable_cfs` flag). Should we do the same for the executor container? - Greg Mann On Feb. 25, 2020, 1:46 a.m., Qian Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71858/ > ----------------------------------------------------------- > > (Updated Feb. 25, 2020, 1:46 a.m.) > > > Review request for mesos, Andrei Budnik and Greg Mann. > > > Bugs: MESOS-10046 > https://issues.apache.org/jira/browse/MESOS-10046 > > > Repository: mesos > > > Description > ------- > > Set resource limits when launching executor container. > > > Diffs > ----- > > src/slave/slave.hpp 03279db082bdba23dbfeb2d93081a908e609aec2 > src/slave/slave.cpp cce275a504effae7a6b71dd333ce8a300d1ce5be > > > Diff: https://reviews.apache.org/r/71858/diff/10/ > > > Testing > ------- > > > Thanks, > > Qian Zhang > >
