----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29915/ -----------------------------------------------------------
(Updated Jan. 15, 2015, 1:32 p.m.) Review request for Aurora and Bill Farner. Changes ------- Revise patch based on offline conversation with wfarner. Summary (updated) ----------------- Ensure TaskStateChange event emitted by StateManagerImpl has valid data. Bugs: AURORA-1016 https://issues.apache.org/jira/browse/AURORA-1016 Repository: aurora Description (updated) ------- The `StateManagerImpl` class emits a pubsub event when a task transitions to the ASSIGNED state but before it attaches the associated slave id and host. This patch ensures that the pubsub event has the associated slave id and host when emitted. Diffs (updated) ----- src/main/java/org/apache/aurora/scheduler/state/StateManagerImpl.java 085454c63a3023e28ae1d148c64d25cfed99c4f1 src/test/java/org/apache/aurora/scheduler/state/StateManagerImplTest.java cdd29ea2b6fc92b967571028d299260556e16d42 Diff: https://reviews.apache.org/r/29915/diff/ Testing ------- ./gradlew build -Pq Thanks, Zameer Manji