aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <wfar...@apache.org>
Subject Re: Review Request 27309: Make executor overhead configurable via CLI.
Date Tue, 28 Oct 2014 22:22:23 GMT


> On Oct. 28, 2014, 10:14 p.m., Joshua Cohen wrote:
> > src/main/java/org/apache/aurora/scheduler/ResourceSlot.java, line 36
> > <https://reviews.apache.org/r/27309/diff/1/?file=735736#file735736line36>
> >
> >     Is there a jira w/ more context? If not can there be? It's not clear to me from
looking at this why this should be removed and what it should be replaced with.

Ah - yes, this should contain a reference to a ticket that's part of the yet-to-be-created
deprecations epic for 0.8.0.  Can you create the 0.8.0 ticket chain?

0.8.0 release candidate (e.g. AURORA-872) blocked by 0.8.0 deprecations epic (e.g. AURORA-423)
containing this new ticket.


- Bill


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


On Oct. 28, 2014, 8:57 p.m., Zameer Manji wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27309/
> -----------------------------------------------------------
> 
> (Updated Oct. 28, 2014, 8:57 p.m.)
> 
> 
> Review request for Aurora and Bill Farner.
> 
> 
> Bugs: AURORA-830
>     https://issues.apache.org/jira/browse/AURORA-830
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Make executor overhead configurable via CLI.
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/aurora/scheduler/MesosTaskFactory.java 83d0406a8bc7ccc1ae29804d2a4c8e8dfb90072c

>   src/main/java/org/apache/aurora/scheduler/ResourceSlot.java ed60447c798a97daceda4a3bba6ee9bcdcaedd0f

>   src/test/java/org/apache/aurora/scheduler/MesosTaskFactoryImplTest.java e96974764844b5d1a3a05f6996075fccee209594

> 
> Diff: https://reviews.apache.org/r/27309/diff/
> 
> 
> Testing
> -------
> 
> ./gradlew clean build -Pq
> 
> 
> Thanks,
> 
> Zameer Manji
> 
>


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