Re: Review Request 26158: Perform job update no-op detection in SchedulerThriftInterface.

2014-09-30 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26158/#review54984 --- Ship it! src/main/java/org/apache/aurora/scheduler/thrift/Schedule

Re: Review Request 26154: Stop sending and receiving DeletedTasks framework message.

2014-09-30 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26154/#review54986 --- Ship it! Ship It! - Maxim Khutornenko On Sept. 30, 2014, 3:36 a.

Re: Review Request 26160: Fixing consumption double counting in quota checks.

2014-09-30 Thread Maxim Khutornenko
> On Sept. 30, 2014, 3:07 a.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/quota/QuotaManager.java, line 249 > > > > > > method name is no longer correct Updated. > On Sept. 30, 2014, 3:07 a.m.

Re: Review Request 26160: Fixing consumption double counting in quota checks.

2014-09-30 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26160/ --- (Updated Sept. 30, 2014, 5:23 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 25835: Serve HTTP assets out of a standard classpath root.

2014-09-30 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25835/#review54996 --- Kevin pointed out in https://issues.apache.org/jira/browse/AURORA-76

Re: Review Request 26158: Perform job update no-op detection in SchedulerThriftInterface.

2014-09-30 Thread Bill Farner
> On Sept. 30, 2014, 4:51 p.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java, > > line 1431 > > > > > > This needs to be OK response. Reverted. - Bill

Re: Review Request 26154: Stop sending and receiving DeletedTasks framework message.

2014-09-30 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26154/#review55013 --- Ping - Brian? - Bill Farner On Sept. 30, 2014, 3:36 a.m., Bill Fa

Re: Review Request 26154: Stop sending and receiving DeletedTasks framework message.

2014-09-30 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26154/#review55015 --- Ship it! Ship It! - Brian Wickman On Sept. 30, 2014, 3:36 a.m.,

Re: Review Request 26158: Perform job update no-op detection in SchedulerThriftInterface.

2014-09-30 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26158/ --- (Updated Sept. 30, 2014, 9:53 p.m.) Review request for Aurora and Maxim Khutorn

Re: Review Request 26158: Perform job update no-op detection in SchedulerThriftInterface.

2014-09-30 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26158/ --- (Updated Sept. 30, 2014, 10:08 p.m.) Review request for Aurora and Maxim Khutor

Re: Review Request 25974: Prevent initial ZK timeouts from killing the executor.

2014-09-30 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25974/ --- (Updated Sept. 30, 2014, 5:17 p.m.) Review request for Aurora, Kevin Sweeney, B