----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28189/#review62065 -----------------------------------------------------------
Bad patch! Reviews applied: [27923] Failed command: ./support/apply-review.sh -n -r 27923 Error: 2014-11-19 00:18:19 URL:https://reviews.apache.org/r/27923/diff/raw/ [366241/366241] -> "27923.patch" [1] error: patch failed: 3rdparty/libprocess/include/process/socket.hpp:46 error: 3rdparty/libprocess/include/process/socket.hpp: patch does not apply error: patch failed: 3rdparty/libprocess/src/clock.cpp:1 error: 3rdparty/libprocess/src/clock.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Nov. 18, 2014, 10:54 p.m., Ian Downes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28189/ > ----------------------------------------------------------- > > (Updated Nov. 18, 2014, 10:54 p.m.) > > > Review request for mesos, Ben Mahler and Vinod Kone. > > > Bugs: MESOS-1718 > https://issues.apache.org/jira/browse/MESOS-1718 > > > Repository: mesos-git > > > Description > ------- > > Use the first task's resources if the ExecutorInfo does not specify any > resources. This avoids the previous slight overcommit. > > I decided to make this review to just removing the overcommit. We'll need to > deprecate the behavior of accepting executors without resources and we should > do the transfer from task to executor for command executors then. Note, as > Ben pointed on the other review, this transfer cannot be done at the slave > level because command executors are not re-registered with the master and we > need to send the true task resources. > > > Diffs > ----- > > src/slave/slave.cpp 06b2e18ff9b202c30f8bf4378cdd35aef734337f > > Diff: https://reviews.apache.org/r/28189/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Ian Downes > >
