----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63094/ -----------------------------------------------------------
(Updated Oct. 19, 2017, 12:10 a.m.) Review request for mesos, Benjamin Bannier, Benjamin Mahler, Chun-Hung Hsiao, Gaston Kleiman, Greg Mann, Joseph Wu, Jan Schlicht, and Vinod Kone. Changes ------- Used UUID and a vector instead. Singular clock idea for the agent does not work. See reasoning in this notes: https://docs.google.com/document/d/1RrrLVATZUyaURpEOeGjgxA6ccshuLo94G678IbL-Yco/edit#bookmark=id.v893d0oct2wh This patch is based on https://reviews.apache.org/r/62903/ Summary (updated) ----------------- Added resource version uuid for offer operations. Repository: mesos Description (updated) ------- The resource version UUID is used to establish the relationship between the operation and the resources that the operation is operating on. Each resource provider will keep a resource version UUID, and changes it when it believes that the resources from this resource provider are out of sync from the master's view. The master will keep track of the last known resource version UUID for each resource provider, and attach the resource version UUID in each operation it sends out. The resource provider should reject operations that have a different resource version UUID than that it maintains, because this means the operation is operating on resources that might have already been invalidated. Diffs (updated) ----- include/mesos/resource_provider/resource_provider.proto f5a9073075327019fd133bd51265f695ef464845 include/mesos/v1/resource_provider/resource_provider.proto e5cbede5b6e57a8641fca1ebfee5454f292cc24c src/messages/messages.proto 0a32b3457e9143a7d48670610ca3e56dd516136f src/tests/resource_provider_manager_tests.cpp ca49e1f0203494fc8b4a4507c33e5a3885a14a59 src/tests/resource_provider_validation_tests.cpp f182bff4670318e9de22c2915c5dbb423a74ad6c Diff: https://reviews.apache.org/r/63094/diff/2/ Changes: https://reviews.apache.org/r/63094/diff/1-2/ Testing ------- make check Thanks, Jie Yu