> On 七月 28, 2016, 11:13 p.m., Benjamin Mahler wrote: > > src/common/resources.cpp, line 458 > > <https://reviews.apache.org/r/50568/diff/1/?file=1456550#file1456550line458> > > > > Hm.. why isn't this just: > > > > ``` > > result.add(resource); > > ```
Sorry, I missed this part when uploading the patch, updated now. Thanks. - Guangya ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50568/#review144028 ----------------------------------------------------------- On 七月 29, 2016, 1:34 a.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50568/ > ----------------------------------------------------------- > > (Updated 七月 29, 2016, 1:34 a.m.) > > > Review request for mesos, Benjamin Mahler, Joris Van Remoortere, and Klaus Ma. > > > Repository: mesos > > > Description > ------- > > This is a follow up action for MESOS-5919. Based on the patch of > https://reviews.apache.org/r/50553/ and > https://reviews.apache.org/r/50557/ , we should update -=/+= to > subtract/add if the resource object is from resources object. > > > Diffs > ----- > > include/mesos/resources.hpp 88a9feabf66ed34e7e5b1c6cb7e831818e7f7883 > include/mesos/v1/resources.hpp 054ed00a03319ae5e350542add34f497eaf79152 > src/common/resources.cpp 3dbff24d6859d3b1ed8589cec50170a5202cfbcb > src/v1/resources.cpp 3c85dc8aa8125962b44e60806ece83a7653d0dc7 > > Diff: https://reviews.apache.org/r/50568/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > Guangya Liu > >
