---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24281/
---
Review request for Aurora, David McLaughlin, Kevin Sweeney, and Bill Farner.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24281/#review49530
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24281/#review49533
---
Ship it!
Ship It!
- Kevin Sweeney
On Aug. 4, 2014, 4:30 p.m.,
On Aug. 4, 2014, 11:34 p.m., Bill Farner wrote:
src/main/java/org/apache/aurora/scheduler/storage/ForwardingStore.java,
line 44
https://reviews.apache.org/r/24281/diff/1/?file=651305#file651305line44
Does this compile?
Maxim Khutornenko wrote:
Everything build and checks
On Aug. 4, 2014, 11:34 p.m., Bill Farner wrote:
src/main/java/org/apache/aurora/scheduler/storage/ForwardingStore.java,
line 44
https://reviews.apache.org/r/24281/diff/1/?file=651305#file651305line44
Does this compile?
Maxim Khutornenko wrote:
Everything build and checks
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24281/#review49537
---
Ship it!
LGTM mod naming
On Aug. 4, 2014, 11:53 p.m., Bill Farner wrote:
src/main/java/org/apache/aurora/scheduler/storage/UpdateStore.java, line 21
https://reviews.apache.org/r/24281/diff/1/?file=651307#file651307line21
Before this continues much futher, i'd love to see s/Update/JobUpdate/
to