{"push_id":2756777718,"size":1,"distinct_size":1,"ref":"refs/heads/v1.0.6-storm-mesos","head":"c5d1bf7531ed1f5addcf15028018d91a34524390","before":"622050ec266e645ae8768b1c6f2eec0c58fae25a","commits":[{"sha":"c5d1bf7531ed1f5addcf15028018d91a34524390","author":{"name":"Erik Weathers","email":"872d470aa32a78f7500825688ecd953376bed386@gmail.com"},"message":"Ensure that ISupervisor.killedWorker() is always called\n\nThis is done by noting if we have transitioned back to EMPTY from another state.\n\nSpecifically, without this change there is a path of states that doesn't lead through `killContainerForChangedAssignment()`, and thus never called `ISupervisor.killedWorker()`.   Here is that path:\n\n```\nEMPTY ->\nWAITING_FOR_BASIC_LOCALIZATION ->\nWAITING_FOR_BLOB_LOCALIZATION ->\nWAITING_FOR_WORKER_START ->\nEMPTY\n```\n\nNote that this never transitions to `KILL` state.\n\nThere are other ways to accomplish this, such as adding 1 or more states into the state machine.  We may readdress this issue at another time.","distinct":true,"url":"https://api.github.com/repos/erikdw/storm/commits/c5d1bf7531ed1f5addcf15028018d91a34524390"}]}
{"actor":{"display_login":"erikdw"}}