---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28449/#review63560
---
Ship it!
Ship It!
- Jake Farrell
On Nov. 25, 2014, 9 p.m., Bill
On Dec. 2, 2014, 2:35 a.m., Maxim Khutornenko wrote:
src/main/java/org/apache/aurora/scheduler/async/preemptor/LiveClusterState.java,
line 71
https://reviews.apache.org/r/28572/diff/1/?file=779239#file779239line71
Please, drop the now unused TASK_TO_SLAVE_ID.
Thanks, made a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28572/
---
(Updated Dec. 2, 2014, 7:04 p.m.)
Review request for Aurora, Kevin Sweeney and
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28572/#review63562
---
Ship it!
Master (d476e2a) is green with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28605/
---
Review request for Aurora, David McLaughlin and Bill Farner.
Bugs: AURORA-952
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28605/#review63563
---
Ship it!
Thanks!!
- Bill Farner
On Dec. 2, 2014, 7:20 p.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28605/#review63564
---
Ship it!
Master (d476e2a) is green with this patch.
On Dec. 2, 2014, 7:30 p.m., Kevin Sweeney wrote:
src/main/java/org/apache/aurora/scheduler/async/preemptor/LiveClusterState.java,
line 67
https://reviews.apache.org/r/28572/diff/2/?file=780331#file780331line67
Citation needed? Iterables.transform generates a lazy view.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28572/#review63574
---
Ship it!
Ship It!
- Kevin Sweeney
On Dec. 2, 2014, 11:04 a.m.,
On Dec. 2, 2014, 11:30 a.m., Kevin Sweeney wrote:
src/main/java/org/apache/aurora/scheduler/async/preemptor/LiveClusterState.java,
line 67
https://reviews.apache.org/r/28572/diff/2/?file=780331#file780331line67
Citation needed? Iterables.transform generates a lazy view.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28607/
---
Review request for Aurora, Maxim Khutornenko and Zameer Manji.
Bugs:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28486/#review63587
---
src/main/python/apache/aurora/executor/aurora_executor.py
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28607/#review63596
---
Ship it!
Master (c86ca6d) is green with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28610/
---
Review request for Aurora, Joshua Cohen and Brian Wickman.
Bugs: AURORA-617
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28610/#review63599
---
3rdparty/python/BUILD
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28610/#review63600
---
Master (c86ca6d) is green with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28610/#review63601
---
Ship it!
wfm, modulo zameer's comment.
- Brian Wickman
On Dec.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28610/#review63603
---
Ship it!
Ship It!
- Joshua Cohen
On Dec. 2, 2014, 10:32 p.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28607/#review63598
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28610/
---
(Updated Dec. 2, 2014, 3:09 p.m.)
Review request for Aurora, Joshua Cohen and
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28613/
---
Review request for Aurora and Bill Farner.
Repository: aurora
Description
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28613/#review63608
---
Ship it!
Master (552ae93) is green with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28617/
---
Review request for Aurora, Kevin Sweeney and Bill Farner.
Bugs: AURORA-909
On Dec. 2, 2014, 11:09 p.m., Maxim Khutornenko wrote:
src/main/java/org/apache/aurora/scheduler/async/preemptor/CachedClusterState.java,
line 52
https://reviews.apache.org/r/28607/diff/1/?file=780416#file780416line52
minor nit: you might want to have it outside the synchronized
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28613/#review63620
---
Ship it!
This is awesome thanks!!
- Bill Farner
On Dec. 2,
On Dec. 2, 2014, 11:09 p.m., Maxim Khutornenko wrote:
src/main/java/org/apache/aurora/scheduler/async/preemptor/CachedClusterState.java,
line 58
https://reviews.apache.org/r/28607/diff/1/?file=780416#file780416line58
Would it make sense to do it conditionally, i.e.:
```java
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28617/#review63627
---
Ship it!
Master (552ae93) is green with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28621/
---
Review request for Aurora and Bill Farner.
Bugs: AURORA-956
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28621/#review63628
---
Ship it!
On Dec. 2, 2014, 11:09 p.m., Maxim Khutornenko wrote:
src/main/java/org/apache/aurora/scheduler/async/preemptor/CachedClusterState.java,
line 58
https://reviews.apache.org/r/28607/diff/1/?file=780416#file780416line58
Would it make sense to do it conditionally, i.e.:
```java
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28607/
---
(Updated Dec. 3, 2014, 1:39 a.m.)
Review request for Aurora, Maxim Khutornenko
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28621/
---
(Updated Dec. 2, 2014, 5:43 p.m.)
Review request for Aurora and Bill Farner.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28621/
---
(Updated Dec. 2, 2014, 5:45 p.m.)
Review request for Aurora and Bill Farner.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28607/#review63632
---
Ship it!
Master (552ae93) is green with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28621/#review63633
---
Ship it!
Master (552ae93) is green with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28621/#review63634
---
Ship it!
This is now on master:
```
$ git log -1 origin/master
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28607/#review63637
---
Ship it!
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28623/
---
Review request for Aurora, David McLaughlin and Kevin Sweeney.
Bugs:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28623/#review63638
---
Ship it!
Ship It!
- Maxim Khutornenko
On Dec. 3, 2014, 2:13
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28623/#review63639
---
Ship it!
Master (8959b49) is green with this patch.
40 matches
Mail list logo