Re: Review Request 25257: Add a separate main class that runs the scheduler in local mode.

2014-09-03 Thread Bill Farner
On Sept. 3, 2014, 1:40 a.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java, line 175 https://reviews.apache.org/r/25257/diff/2/?file=674608#file674608line175 Can this TODO be dropped now? Sure, done. - Bill

Re: Review Request 25257: Add a separate main class that runs the scheduler in local mode.

2014-09-02 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25257/ --- (Updated Sept. 2, 2014, 11:39 p.m.) Review request for Aurora, Joshua Cohen,

Re: Review Request 25257: Add a separate main class that runs the scheduler in local mode.

2014-09-02 Thread Joshua Cohen
On Sept. 2, 2014, 6:31 p.m., Joshua Cohen wrote: src/test/java/org/apache/aurora/scheduler/app/local/FakeMaster.java, lines 134-135 https://reviews.apache.org/r/25257/diff/1/?file=674033#file674033line134 Any way to avoid magic numbers here (and would it be worthwhile)? As a