----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25357/#review52362 -----------------------------------------------------------
Ship it! lgtm(y still scheduler-naive eyes). src/main/java/org/apache/aurora/scheduler/TaskVars.java <https://reviews.apache.org/r/25357/#comment91130> use host (declared above) here in place of task.getAssignedTask().getSlaveHost()? - Joshua Cohen On Sept. 4, 2014, 9:42 p.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25357/ > ----------------------------------------------------------- > > (Updated Sept. 4, 2014, 9:42 p.m.) > > > Review request for Aurora, David McLaughlin and Bill Farner. > > > Bugs: AURORA-685 > https://issues.apache.org/jira/browse/AURORA-685 > > > Repository: aurora > > > Description > ------- > > Currently tracking LOST and FAILED states. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/TaskVars.java > 6654c1675ac9f5f7d481e115cea7c224fb212467 > src/test/java/org/apache/aurora/scheduler/TaskVarsTest.java > d02714c846a521ff9ac3e53d991731314e714ae2 > > Diff: https://reviews.apache.org/r/25357/diff/ > > > Testing > ------- > > gradle -Pq build > > > Thanks, > > Maxim Khutornenko > >