---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32220/
---
(Updated March 19, 2015, 11:39 p.m.)
Review request for Aurora, Bill Farner
On March 19, 2015, 9:23 p.m., Bill Farner wrote:
src/main/java/org/apache/aurora/scheduler/async/preemptor/PreemptionSlotCache.java,
line 45
https://reviews.apache.org/r/32220/diff/2/?file=899548#file899548line45
This should refer to the parameter, not the guice binding
On March 19, 2015, 9:41 p.m., Zameer Manji wrote:
src/main/java/org/apache/aurora/scheduler/async/preemptor/PreemptionSlotFinder.java,
line 254
https://reviews.apache.org/r/32220/diff/2/?file=899549#file899549line254
This is a bit confusing to see .getSlaveId and then construct
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32220/#review77139
---
Ship it!
Master (6396410) is green with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32220/#review77106
---
Ship it!
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32220/#review77112
---
Ship it!
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32220/
---
(Updated March 19, 2015, 12:29 a.m.)
Review request for Aurora, Bill Farner
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32220/#review76986
---
Ship it!
Master (6396410) is green with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32220/#review76969
---
Ship it!
Master (6396410) is green with this patch.
