Re: Review Request 25398: Make the offer hold jitter window configurable.

2014-09-08 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25398/#review52613 --- src/main/java/org/apache/aurora/scheduler/async/AsyncModule.java

Re: Review Request 25398: Make the offer hold jitter window configurable.

2014-09-08 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25398/ --- (Updated Sept. 8, 2014, 5:54 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 25398: Make the offer hold jitter window configurable.

2014-09-08 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25398/ --- (Updated Sept. 8, 2014, 5:55 p.m.) Review request for Aurora, Maxim

Review Request 25450: Fix broken large update warning.

2014-09-08 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25450/ --- Review request for Aurora, David McLaughlin and Maxim Khutornenko. Bugs:

Re: Review Request 25450: Fix broken large update warning.

2014-09-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25450/#review52624 --- src/main/python/apache/aurora/client/cli/jobs.py

Re: Review Request 25398: Make the offer hold jitter window configurable.

2014-09-08 Thread Joshua Cohen
On Sept. 8, 2014, 8:41 p.m., Maxim Khutornenko wrote: src/test/java/org/apache/aurora/scheduler/async/RandomJitterReturnDelayTest.java, line 71 https://reviews.apache.org/r/25398/diff/2/?file=682734#file682734line71 How about a assertRandomJitterReturnDelay(0, 0) test? Added.

Re: Review Request 25398: Make the offer hold jitter window configurable.

2014-09-08 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25398/ --- (Updated Sept. 8, 2014, 9:09 p.m.) Review request for Aurora, Maxim

Review Request 25455: Deprecating PopulateJobResult.populated thrift field.

2014-09-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25455/ --- Review request for Aurora, Bill Farner and Zameer Manji. Repository: aurora

Re: Review Request 25455: Deprecating PopulateJobResult.populated thrift field.

2014-09-08 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25455/#review52642 --- Ship it! Ship It! - Zameer Manji On Sept. 8, 2014, 2:46 p.m.,

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/ --- Review request for Aurora, David McLaughlin and Bill Farner. Bugs: AURORA-649

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

Review Request 25259: Add update information to the scheduler UI

2014-09-08 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25259/ --- Review request for Aurora, Joshua Cohen, Kevin Sweeney, and Bill Farner. Bugs:

Review Request 25466: Upgrade Aurora to psutil 2.1.1

2014-09-08 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25466/ --- Review request for Aurora, Joe Smith and Kevin Sweeney. Bugs: AURORA-412

Re: Review Request 25466: Upgrade Aurora to psutil 2.1.1

2014-09-08 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25466/#review52662 --- Ship it! src/main/python/apache/thermos/core/helper.py

Re: Review Request 25259: Add update information to the scheduler UI

2014-09-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25259/#review52661 ---

Re: Review Request 25259: Add update information to the scheduler UI

2014-09-08 Thread Maxim Khutornenko
On Sept. 9, 2014, 1:45 a.m., Maxim Khutornenko wrote: src/main/resources/org/apache/aurora/scheduler/http/ui/js/controllers.js, line 277 https://reviews.apache.org/r/25259/diff/1/?file=683446#file683446line277 This seems to be too short to make any visual difference but could be

Re: Review Request 25466: Upgrade Aurora to psutil 2.1.1

2014-09-08 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25466/ --- (Updated Sept. 9, 2014, 2:13 a.m.) Review request for Aurora, Joe Smith and

Re: Review Request 25455: Deprecating PopulateJobResult.populated thrift field.

2014-09-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25455/#review52671 --- Ship it! - Bill Farner On Sept. 8, 2014, 9:46 p.m., Maxim

Re: Review Request 25466: Upgrade Aurora to psutil 2.1.1

2014-09-08 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25466/#review52674 --- Ship it! awesome - Joe Smith On Sept. 8, 2014, 7:13 p.m., Brian