Re: Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-31 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27371/#review59412 --- Ship it! Master (02a1c04) is green with this patch. ./build-suppo

Re: Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-31 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27371/ --- (Updated Oct. 31, 2014, 8:31 p.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-31 Thread Bill Farner
> On Oct. 30, 2014, 5:29 p.m., Kevin Sweeney wrote: > > buildSrc/src/main/groovy/org/apache/aurora/CoverageReportCheck.groovy, line > > 89 > > > > > > looks like your sentence got cut off here - the > > legacyClassesW

Re: Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-31 Thread Bill Farner
> On Oct. 30, 2014, 3:48 p.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/async/AsyncModule.java, line 212 > > > > > > s/public// Done. I also realized the constructor injection was a hack

Re: Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-31 Thread Bill Farner
> On Oct. 30, 2014, 5 p.m., Joshua Cohen wrote: > > src/test/java/org/apache/aurora/scheduler/async/AsyncModuleTest.java, line > > 58 > > > > > > Each test is invoking this method, then accessing the injector instance

Re: Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-30 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27371/#review59232 --- Ship it! buildSrc/src/main/groovy/org/apache/aurora/CoverageReport

Re: Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-30 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27371/#review59224 --- Ship it! src/test/java/org/apache/aurora/scheduler/async/AsyncModu

Re: Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-30 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27371/#review59206 --- Ship it! src/main/java/org/apache/aurora/scheduler/async/AsyncModu

Re: Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-29 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27371/#review59140 --- Ship it! Master (d85e616) is green with this patch. ./build-suppo

Review Request 27371: Add stat gauge coverage for AsyncModule and MemStorage.

2014-10-29 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27371/ --- Review request for Aurora, Joshua Cohen and Maxim Khutornenko. Bugs: AURORA-884