Re: Review Request 17131: Improve test coverage for CronJobManager.

2014-01-22 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17131/ --- (Updated Jan. 22, 2014, 7:02 p.m.) Review request for Aurora, Suman Karumuri an

Re: Review Request 17131: Improve test coverage for CronJobManager.

2014-01-22 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17131/ --- (Updated Jan. 22, 2014, 6:56 p.m.) Review request for Aurora, Suman Karumuri an

Re: Review Request 17131: Improve test coverage for CronJobManager.

2014-01-22 Thread Bill Farner
> On Jan. 21, 2014, 3:29 p.m., Suman Karumuri wrote: > > src/main/java/org/apache/aurora/scheduler/state/CronJobManager.java, line > > 456 > > > > > > The name CronJob sounds too generic. Consider changing it to > >

Re: Review Request 17131: Improve test coverage for CronJobManager.

2014-01-22 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17131/#review32516 --- Ship it! Ship It! - Suman Karumuri On Jan. 20, 2014, 9:01 p.m.,

Re: Review Request 17131: Improve test coverage for CronJobManager.

2014-01-21 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17131/#review32376 --- Ship it! src/main/java/org/apache/aurora/scheduler/state/CronJobMa

Re: Review Request 17131: Improve test coverage for CronJobManager.

2014-01-21 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17131/#review32365 --- src/main/java/org/apache/aurora/scheduler/state/CronJobManager.java

Review Request 17131: Improve test coverage for CronJobManager.

2014-01-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17131/ --- Review request for Aurora, Suman Karumuri and Maxim Khutornenko. Bugs: AURORA-6