> On Jan. 4, 2016, 4:21 p.m., Guangya Liu wrote: > > src/slave/slave.cpp, line 3954 > > <https://reviews.apache.org/r/41857/diff/1/?file=1180472#file1180472line3954> > > > > Here should be requested.stateless().reserved()
Using `allocationSlackable` for `stateless().reserved()`. > On Jan. 4, 2016, 4:21 p.m., Guangya Liu wrote: > > src/slave/slave.cpp, line 4008 > > <https://reviews.apache.org/r/41857/diff/1/?file=1180472#file1180472line4008> > > > > What about using following to get evictable executors? > > > > if (availabe-occupied+evicting).contains(reserved) > > wait for executor terminated future > > else: > > evcit some executors `available` may not contains `occupied`. - Klaus ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41857/#review112516 ----------------------------------------------------------- On Jan. 9, 2016, 5:06 p.m., Klaus Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41857/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2016, 5:06 p.m.) > > > Review request for mesos, Ben Mahler, Guangya Liu, Artem Harutyunyan, Joris > Van Remoortere, Joseph Wu, and Jian Qiu. > > > Bugs: MESOS-3892 > https://issues.apache.org/jira/browse/MESOS-3892 > > > Repository: mesos > > > Description > ------- > > get evictable executors > > > Diffs > ----- > > src/slave/slave.hpp b7586ce42bfac9d9885a3eb8d82deb94680c236c > src/slave/slave.cpp 90d0fecd2d83fd174134870a577ac59d79c0006f > > Diff: https://reviews.apache.org/r/41857/diff/ > > > Testing > ------- > > > Thanks, > > Klaus Ma > >
