----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24486/#review49992 -----------------------------------------------------------
Ship it! src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java <https://reviews.apache.org/r/24486/#comment87448> LOG.warning() here and below? - Maxim Khutornenko On Aug. 8, 2014, 12:42 a.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24486/ > ----------------------------------------------------------- > > (Updated Aug. 8, 2014, 12:42 a.m.) > > > Review request for Aurora and Kevin Sweeney. > > > Bugs: AURORA-641 > https://issues.apache.org/jira/browse/AURORA-641 > > > Repository: aurora > > > Description > ------- > > Drop HostAttributes with no slaveId when recovering from persistent storage. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java > 345442035ccfa0a04a92f9eb389add50d0a492d9 > > src/main/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImpl.java > 5d3ec36d9907be543296847134c53e6ddc6edc70 > src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java > a9545e1656a3b1b0ae55155eec4f06878158d8da > > src/test/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImplTest.java > 48600b59b918434ef963b758279c48584ccd34ef > > Diff: https://reviews.apache.org/r/24486/diff/ > > > Testing > ------- > > ./gradlew build -Pq > > > Thanks, > > Bill Farner > >
