Re: Review Request 37049: Integrating DelayExecutor into the scheduler's transaction handling.

2015-08-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37049/#review93983 --- Ship it! Master (7fdcfef) is green with this patch.

Review Request 37049: Integrating DelayExecutor into the scheduler's transaction handling.

2015-08-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37049/ --- Review request for Aurora and Kevin Sweeney. Bugs: AURORA-1395

Review Request 37041: Remove defunct scheduler-local.sh.

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

Re: Review Request 36972: One python_library per exported setup.py project

2015-08-03 Thread Kevin Sweeney
On July 31, 2015, 3:21 p.m., Brian Wickman wrote: I think it makes sense to split into two binary-exporting packages: one client-side and one server-side. The client side (aurora.client) should contain aurora client and aurora admin client. The server-side (reuse aurora.executor?)

Re: Review Request 36972: One python_library per exported setup.py project

2015-08-03 Thread Brian Wickman
On July 31, 2015, 10:21 p.m., Brian Wickman wrote: I think it makes sense to split into two binary-exporting packages: one client-side and one server-side. The client side (aurora.client) should contain aurora client and aurora admin client. The server-side (reuse aurora.executor?)

Re: Review Request 36972: One python_library per exported setup.py project

2015-08-03 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93932 --- Ship it! - Brian Wickman On Aug. 1, 2015, 10:38 p.m., Kevin

Re: Review Request 37001: Centralizing offer/task matching in TaskAssigner.

2015-08-03 Thread Maxim Khutornenko
On Aug. 3, 2015, 5:38 p.m., Zameer Manji wrote: src/main/java/org/apache/aurora/scheduler/scheduling/TaskScheduler.java, line 111 https://reviews.apache.org/r/37001/diff/1/?file=1026645#file1026645line111 nit: Was it needed to flip the BiCache type here? If not, it would be

Re: Review Request 37001: Centralizing offer/task matching in TaskAssigner.

2015-08-03 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37001/#review93922 --- Ship it!

Re: Review Request 36972: One python_library per exported setup.py project

2015-08-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36972/#review93930 --- Ship it! Ship It! - Bill Farner On Aug. 1, 2015, 10:38 p.m.,