----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70788/ -----------------------------------------------------------
(Updated June 6, 2019, 6:35 p.m.) Review request for mesos and Benjamin Bannier. Changes ------- Addressed Benjamin's comment. Bugs: MESOS-9831 https://issues.apache.org/jira/browse/MESOS-9831 Repository: mesos Description ------- Previously when there is a missing resource provider in an `UpdateSlaveMessage` but the agent's total resources remain unchanged, the update message will be completely ignored, so the missing resource provider will still be cached in the master's state, which is not the desired behavior. This patch ensures that the master's state gets updated if any resource provider is missing. Diffs (updated) ----- src/master/master.cpp 4d7c37cf8814e45432b3fe15173f5343676a372b src/tests/api_tests.cpp 37d0cb11ddb24b32f1e83feefe168f2aa2eb659c Diff: https://reviews.apache.org/r/70788/diff/2/ Changes: https://reviews.apache.org/r/70788/diff/1-2/ Testing ------- make check Thanks, Chun-Hung Hsiao