Re: Review Request 27712: Fixing beta-update OK status messaging.

2014-11-07 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27712/#review60352 --- Ship it! LGTM once Job update has started message is removed from

Re: Review Request 27712: Fixing beta-update OK status messaging.

2014-11-07 Thread Maxim Khutornenko
On Nov. 7, 2014, 6:05 p.m., Bill Farner wrote: src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java, line 1582 https://reviews.apache.org/r/27712/diff/1/?file=754027#file754027line1582 Please remove. The API really should not be sending back messages of

Re: Review Request 27712: Fixing beta-update OK status messaging.

2014-11-07 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27712/ --- (Updated Nov. 7, 2014, 7:12 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 27712: Fixing beta-update OK status messaging.

2014-11-07 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27712/ --- (Updated Nov. 7, 2014, 7:13 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 27712: Fixing beta-update OK status messaging.

2014-11-06 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27712/#review60282 --- Ship it! Master (5250a94) is green with this patch.