----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69458/ -----------------------------------------------------------
Review request for mesos, Chun-Hung Hsiao, Gastón Kleiman, Greg Mann, and Jan Schlicht. Repository: mesos Description ------- When the agent handles an `UpdateOperationStatusMessage` from a resource provider, it injects its own ID which is (at least conceptually) unknown to the resource provider before forwarding the message to the master, and also updates its own tracking for the operation. This patch makes sure that we first mutate the message before handing it on for updating the internal operation tracking, while previously we used the unmodified message. Always using the same message reduces error potential if in future changes we e.g., introduce agent operation status update managers. Diffs ----- src/slave/slave.cpp 858b78620e1ef33f3587d0bd95a684917aaf5bbb Diff: https://reviews.apache.org/r/69458/diff/1/ Testing ------- `make check` Thanks, Benjamin Bannier
