> On Nov. 11, 2014, 2:34 p.m., Bill Farner wrote:
> > src/test/java/org/apache/aurora/scheduler/async/KillRetryTest.java, line 103
> > <https://reviews.apache.org/r/27757/diff/1/?file=755113#file755113line103>
> >
> >     remove

fixed.


> On Nov. 11, 2014, 2:34 p.m., Bill Farner wrote:
> > src/main/java/org/apache/aurora/scheduler/SchedulerServicesModule.java, 
> > line 74
> > <https://reviews.apache.org/r/27757/diff/1/?file=755101#file755101line74>
> >
> >     remove comment

Done.


> On Nov. 11, 2014, 2:34 p.m., Bill Farner wrote:
> > src/main/java/org/apache/aurora/scheduler/async/AsyncModule.java, line 251
> > <https://reviews.apache.org/r/27757/diff/1/?file=755103#file755103line251>
> >
> >     revert

Fixed.


> On Nov. 11, 2014, 2:34 p.m., Bill Farner wrote:
> > src/main/java/org/apache/aurora/scheduler/SchedulerServicesModule.java, 
> > line 72
> > <https://reviews.apache.org/r/27757/diff/1/?file=755101#file755101line72>
> >
> >     How about fully untethering here and doing this work in `SchedulerMain`?

nowhere immediately jumps out as the right place to put this and I'd like to 
minimize the ripple here.


- Kevin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27757/#review60874
-----------------------------------------------------------


On Nov. 7, 2014, 4:06 p.m., Kevin Sweeney wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27757/
> -----------------------------------------------------------
> 
> (Updated Nov. 7, 2014, 4:06 p.m.)
> 
> 
> Review request for Aurora and Bill Farner.
> 
> 
> Bugs: AURORA-920
>     https://issues.apache.org/jira/browse/AURORA-920
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> This attempts to kill LifecycleModule in favor of service registration.
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/aurora/scheduler/AppStartup.java PRE-CREATION 
>   src/main/java/org/apache/aurora/scheduler/SchedulerModule.java 
> 72d3d60c602b2cf01445f64f3be7e4ecf53008ca 
>   src/main/java/org/apache/aurora/scheduler/SchedulerServicesModule.java 
> PRE-CREATION 
>   src/main/java/org/apache/aurora/scheduler/app/AppModule.java 
> fef76f5069a80959eaba04f1b4b93ff3f6ba43dd 
>   src/main/java/org/apache/aurora/scheduler/async/AsyncModule.java 
> 4e37f4c9c8d4cde477a96a9b8cca7a075f170919 
>   src/main/java/org/apache/aurora/scheduler/async/JobUpdateHistoryPruner.java 
> 8e9a1dc10998a313ad92e365c739b74f892e82ce 
>   src/main/java/org/apache/aurora/scheduler/cron/quartz/CronModule.java 
> 22c666e28e03f3c1e5a1d89fda55641526b11162 
>   src/main/java/org/apache/aurora/scheduler/events/PubsubEventModule.java 
> cd5aa17b24209b186314f431c65062d6ed9972cb 
>   src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java 
> 36706f200e245ea08afa4dd36cb3511d0a63b097 
>   src/main/java/org/apache/aurora/scheduler/sla/SlaModule.java 
> 354844a2540dec6450724acca18c399bc6c01f44 
>   src/main/java/org/apache/aurora/scheduler/stats/AsyncStatsModule.java 
> 844a38a285f7abf917a844a7f6a4a14749aa9b57 
>   src/main/java/org/apache/aurora/scheduler/stats/TaskStatCalculator.java 
> a7f9401f70b567002c83f4f56407b4e1d3afa5a5 
>   src/test/java/org/apache/aurora/scheduler/async/AsyncModuleTest.java 
> 962aff8f4fa590935773c9fe90b1a6f59bc1c51f 
>   
> src/test/java/org/apache/aurora/scheduler/async/JobUpdateHistoryPrunerTest.java
>  1376237efc8f109d5a57bb8bff1c05026f257418 
>   src/test/java/org/apache/aurora/scheduler/async/KillRetryTest.java 
> 662ebdceff580af0a05f9a89d50ca1c90ddc9070 
>   src/test/java/org/apache/aurora/scheduler/http/JettyServerModuleTest.java 
> 4d58240234b8e48c1f081a8ae647c54095f9dc1d 
>   src/test/java/org/apache/aurora/scheduler/state/PubsubTestUtil.java 
> e884209fd5965e50847f14b09a43f6aba4946b7e 
> 
> Diff: https://reviews.apache.org/r/27757/diff/
> 
> 
> Testing
> -------
> 
> ./gradlew -Pq build
> 
> 
> Thanks,
> 
> Kevin Sweeney
> 
>

Reply via email to