On May 20, 2014, 10:58 p.m., Bill Farner wrote:
src/test/java/org/apache/aurora/scheduler/thrift/aop/ForwardingThrift.java,
line 135
https://reviews.apache.org/r/21383/diff/4-5/?file=584804#file584804line135
remove trailing WS.
this should be trivial to catch in
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/
---
(Updated May 21, 2014, 9:05 a.m.)
Review request for Aurora, David McLaughlin
On May 21, 2014, 2:58 a.m., Bill Farner wrote:
src/test/java/org/apache/aurora/scheduler/thrift/aop/ForwardingThrift.java,
line 135
https://reviews.apache.org/r/21383/diff/4-5/?file=584804#file584804line135
remove trailing WS.
this should be trivial to catch in
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/#review43535
---
Ship it!
lgtm.
- David McLaughlin
On May 20, 2014, 12:25 a.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/
---
(Updated May 20, 2014, 6:56 p.m.)
Review request for Aurora, David McLaughlin
On May 21, 2014, 2:58 a.m., Bill Farner wrote:
src/test/java/org/apache/aurora/scheduler/thrift/aop/ForwardingThrift.java,
line 135
https://reviews.apache.org/r/21383/diff/4-5/?file=584804#file584804line135
remove trailing WS.
this should be trivial to catch in
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/
---
(Updated May 19, 2014, 11:25 a.m.)
Review request for Aurora, David McLaughlin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/#review43388
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/#review43397
---
On May 19, 2014, 9:11 p.m., Mark Chu-Carroll wrote:
src/main/java/org/apache/aurora/scheduler/state/SchedulerCoreImpl.java,
line 147
https://reviews.apache.org/r/21383/diff/3/?file=584300#file584300line147
Just to be clear: there is no deprecation here. The existing behavior
On May 19, 2014, 5:11 p.m., Mark Chu-Carroll wrote:
src/main/java/org/apache/aurora/scheduler/state/SchedulerCoreImpl.java,
line 147
https://reviews.apache.org/r/21383/diff/3/?file=584300#file584300line147
Just to be clear: there is no deprecation here. The existing behavior
On May 19, 2014, 9:11 p.m., Mark Chu-Carroll wrote:
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java,
line 258
https://reviews.apache.org/r/21383/diff/3/?file=584301#file584301line258
There are tests of scheduleCronJob and descheduleCronJob below.
On May 19, 2014, 9:11 p.m., Mark Chu-Carroll wrote:
src/main/java/org/apache/aurora/scheduler/state/SchedulerCoreImpl.java,
line 147
https://reviews.apache.org/r/21383/diff/3/?file=584300#file584300line147
Just to be clear: there is no deprecation here. The existing behavior
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/
---
(Updated May 19, 2014, 8:25 p.m.)
Review request for Aurora, David McLaughlin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/#review43012
---
Heads up: this has high likelihood of conflicting with
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/
---
(Updated May 14, 2014, 10:39 a.m.)
Review request for Aurora, David McLaughlin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21383/
---
Review request for Aurora, David McLaughlin and Bill Farner.
Bugs: aurora-417
17 matches
Mail list logo