----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71361/#review217426 -----------------------------------------------------------
Ship it! Ship It! - Gilbert Song On Aug. 23, 2019, 5:57 a.m., Andrei Budnik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71361/ > ----------------------------------------------------------- > > (Updated Aug. 23, 2019, 5:57 a.m.) > > > 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 > >