----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69718/ -----------------------------------------------------------
Review request for mesos and Jan Schlicht. Repository: mesos Description ------- The resource provider manager's `start` method takes ownership of a passed `Owned` value. Since `Owned` values are not shared (but owned), they cannot be copied (at least semantically until MESOS-5122 is implemented). This patch fixes existing code to make sure we do not copy `Owned` values when invoking `ResourceProviderManager::start`. This should be a non-functional change modulo existing bugs. Diffs ----- src/tests/api_tests.cpp c597243e2e210e83a4ab7441fbcfa3198b43d849 src/tests/master_slave_reconciliation_tests.cpp 979048c1e90966244135f149ff5ef66054f27d0d src/tests/master_tests.cpp 67713c81953d7ed90385ffbcfd7804de575c8977 src/tests/operation_reconciliation_tests.cpp a5e81a1d50ea6135932610d5e4c45e87e9354a09 src/tests/resource_provider_manager_tests.cpp 20cfb340bf634354865d79c92ee70cddca08f28a src/tests/slave_tests.cpp 4aed5d68e9a408821880ffaede482937be1999f4 Diff: https://reviews.apache.org/r/69718/diff/1/ Testing ------- `make check` `./support/mesos-tidy.sh` Thanks, Benjamin Bannier
