Re: Review Request 25459: Adding quota check into startJobUpdate.

2014-09-12 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25459/#review53238 ---

Re: Review Request 25459: Adding quota check into startJobUpdate.

2014-09-12 Thread Maxim Khutornenko
On Sept. 12, 2014, 11:04 p.m., Bill Farner wrote: src/main/java/org/apache/aurora/scheduler/state/TaskLimitValidator.java, line 102 https://reviews.apache.org/r/25459/diff/3/?file=683405#file683405line102 You mean inclusive? Not really. Neither 0 nor max value makes sense in

Re: Review Request 25459: Adding quota check into startJobUpdate.

2014-09-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25459/ --- (Updated Sept. 9, 2014, 12:04 a.m.) Review request for Aurora, David

Re: Review Request 25459: Adding quota check into startJobUpdate.

2014-09-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25459/ --- (Updated Sept. 9, 2014, 12:30 a.m.) Review request for Aurora, David