----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68639/#review208393 -----------------------------------------------------------
PASS: Mesos patch 68639 was successfully built and tested. Reviews applied: `['68490', '68639']` All the build artifacts available at: http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2300/mesos-review-68639 - Mesos Reviewbot Windows On Sept. 6, 2018, 12:13 a.m., Meng Zhu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68639/ > ----------------------------------------------------------- > > (Updated Sept. 6, 2018, 12:13 a.m.) > > > Review request for mesos, Benjamin Bannier, Benjamin Mahler, and Gastón > Kleiman. > > > Repository: mesos > > > Description > ------- > > Due to the copy-on-write optimization (MESOS-6765), one needs to > check the `use_count` of `Resource_` before mutating. Currently, > there is no mechanism to enforce this. As a short-term mitigation > measure, we rename `resources` to `noMutationWithoutExclusiveOwnership` > to alert people about obtaining an exclusive ownership before mutating > the `Resource_` objects. > > > Diffs > ----- > > include/mesos/resources.hpp 6f81b14f8bc090a144eeae8f15639c370366166d > include/mesos/v1/resources.hpp 09110530da16678abf6bf6b308906dd8ccc8180a > src/common/resources.cpp 3e63cdedb9261970dbeb9bb9f97eed65819f68a7 > src/v1/resources.cpp 3683a331e0859cd6f2ad061db6ba67112ecfcb0d > > > Diff: https://reviews.apache.org/r/68639/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Meng Zhu > >