search

mesos / storm

Storm on Mesos!
Apache License 2.0
139 stars 66 forks source link

improve handling of status updates #156

Closed erikdw closed 8 years ago

erikdw commented 8 years ago
  1. For unrecognized status types we were falling into the default case and doing nothing. That was really bad with the introduction of TASK_ERROR for situations where the ExecutorInfo was mismatched between tasks for a topology, since the update/error wasn't appearing in the nimbus logs, only in the mesos-master logs. So now we log such updates at warning level.
  2. Also just be more explicit about printing out the status updates for the normal flow, by logging at info level for TASK_RUNNING and the terminal states.
  3. Also rename the taskLost method to taskTerminated, since it is not just for the lost statuses that we perform the action.
JessicaLHartog commented 8 years ago

👍 LGTM