----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65338/#review196275 -----------------------------------------------------------
Ship it! Ship It! - Jordan Ly On Jan. 25, 2018, 6:21 p.m., David McLaughlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65338/ > ----------------------------------------------------------- > > (Updated Jan. 25, 2018, 6:21 p.m.) > > > Review request for Aurora, Jordan Ly and Santhosh Kumar Shanmugham. > > > Bugs: AURORA-1965 > https://issues.apache.org/jira/browse/AURORA-1965 > > > Repository: aurora > > > Description > ------- > > Discovered while debugging https://issues.apache.org/jira/browse/AURORA-1966. > Before we attempt to launch a task, we move the task to ASSIGNED state. > However, the code to deal with launch failures expects the task to be in > PENDING state. So the ASSIGNED -> LOST state change fails, and instead we > rely on the transient task timeout for correctness. This means errors that > can be recovered from in seconds instead take at least five minutes (by > default). > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/scheduling/TaskAssignerImpl.java > 916908bbf635a261c01777cd3a357ca457dd9726 > > src/test/java/org/apache/aurora/scheduler/scheduling/TaskAssignerImplTest.java > 533bb44953163e2148fa18c394a4338938dae205 > > > Diff: https://reviews.apache.org/r/65338/diff/1/ > > > Testing > ------- > > ./gradlew test > > Also tested in Vagrant. > > > Thanks, > > David McLaughlin > >