mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Klaus Ma" <klaus1982...@gmail.com>
Subject Re: Review Request 41857: Got evictable executors.
Date Sat, 09 Jan 2016 09:06:57 GMT


> On Jan. 8, 2016, 4:36 p.m., Jian Qiu wrote:
> > src/slave/slave.cpp, line 3985
> > <https://reviews.apache.org/r/41857/diff/3/?file=1187189#file1187189line3985>
> >
> >     I am not sure whether this situation will happen, because it means a task using
allocation slack and is waiting for evicting executor. Master should not allowed such task
to be launched beacause there not enough available allocation slack resources for this task

Good catch!


- Klaus


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41857/#review113430
-----------------------------------------------------------


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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message