> On Oct. 26, 2017, 11:08 p.m., Greg Mann wrote: > > src/messages/messages.proto > > Lines 712 (patched) > > <https://reviews.apache.org/r/63094/diff/3/?file=1864433#file1864433line712> > > > > Do we need multiple ResourceVersionUUIDs here? Can the master just > > include the UUID for the relevant resource provider/agent?
Yeah, good point. - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63094/#review189371 ----------------------------------------------------------- On Oct. 19, 2017, 9:35 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63094/ > ----------------------------------------------------------- > > (Updated Oct. 19, 2017, 9:35 p.m.) > > > Review request for mesos, Benjamin Bannier, Benjamin Mahler, Chun-Hung Hsiao, > Gaston Kleiman, Greg Mann, Joseph Wu, Jan Schlicht, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > 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 > ----- > > 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/3/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >