----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65482/ -----------------------------------------------------------
(Updated Feb. 14, 2018, 3:21 p.m.) Review request for mesos, Greg Mann, Jie Yu, and Jan Schlicht. Summary (updated) ----------------- Improved handling of non-terminal operations after master failover. Bugs: MESOS-8536 https://issues.apache.org/jira/browse/MESOS-8536 Repository: mesos Description (updated) ------- This patch fixes the handling of non-terminal operations learned by a newly elected master after a master failover, so that only these operations are counted as using resources. Previously we did not count any operations as using resources which by accident produced expected behavior if the operation was already terminal when the master learned about them. We do not address the issue of being unable to properly account for operations triggered by frameworks unknown to the master, see MESOS-8582. Instead we emit a warning for now since the master might continue to abort due to assertion failures due to incomplete resource accounting. Diffs (updated) ----- src/master/master.cpp d7d22866f7a4eb87bd8949efafc97e828e7d4b94 Diff: https://reviews.apache.org/r/65482/diff/2/ Changes: https://reviews.apache.org/r/65482/diff/1-2/ Testing ------- `make check`, also tested with a version of the test added in r/65045 which triggered this issue. Thanks, Benjamin Bannier