----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71361/ -----------------------------------------------------------
Review request for mesos, Gilbert Song, Greg Mann, and Qian Zhang. Repository: mesos Description ------- Previously, if `statusUpdate` was called for a pending task, it would forward the status update and then continue executing `statusUpdate`, which then checks if there is an executor that is aware of this task. Given that a pending task is not known to any executor, it would always handle it by forwarding status update one more time. This patch adds missing `return` statement, which fixes the issue. Diffs ----- src/slave/slave.cpp 1d0ec9d2428c3ffa28ad3e960b74f171013cf0c2 Diff: https://reviews.apache.org/r/71361/diff/1/ Testing ------- `sudo make check` Thanks, Andrei Budnik