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 39401: Quota: Updated allocate() in the hierarchical allocator to support quota.
Date Sat, 07 Nov 2015 13:19:17 GMT


> On Oct. 26, 2015, 10:07 p.m., Klaus Ma wrote:
> > src/master/allocator/mesos/hierarchical.cpp, line 1004
> > <https://reviews.apache.org/r/39401/diff/5/?file=1105060#file1105060line1004>
> >
> >     Just check the code, dynamically reserved resource are included in allocation.
> 
> Qian Zhang wrote:
>     Really?:-) I think once a resource is dynamically reserved, it will be removed from
the framework's allocation and marked as reserved in slave's total resources, so that in next
allocation cycle it can be offered to the framework of the role which reserves it.
> 
> Alexander Rukletsov wrote:
>     A reserved resource can be allocated or not. For example, if a framework declines
a reserved resource, it will be removed from allocated.

Thanks, @AlexR; got your point.


- Klaus


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


On Nov. 6, 2015, 2:25 a.m., Alexander Rukletsov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39401/
> -----------------------------------------------------------
> 
> (Updated Nov. 6, 2015, 2:25 a.m.)
> 
> 
> Review request for mesos, Bernd Mathiske, Joerg Schad, Joris Van Remoortere, and Joseph
Wu.
> 
> 
> Bugs: MESOS-3718
>     https://issues.apache.org/jira/browse/MESOS-3718
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   src/master/allocator/mesos/hierarchical.cpp f4e4a123d3da0442e8b0b0ad14d1ee760752ba36

> 
> Diff: https://reviews.apache.org/r/39401/diff/
> 
> 
> Testing
> -------
> 
> make check (Mac OS X 10.10.4)
> 
> 
> Thanks,
> 
> Alexander Rukletsov
> 
>


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