----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48940/#review139409 -----------------------------------------------------------
src/master/master.hpp (line 1103) <https://reviews.apache.org/r/48940/#comment204635> I think I missed this in the previous review, but lets call this `get` as well. Do this in a different review. src/master/weights_handler.cpp <https://reviews.apache.org/r/48940/#comment204636> why this change? looks like we want a sanitized role (#193) to be inside validatedWeightInfos. src/master/weights_handler.cpp (line 216) <https://reviews.apache.org/r/48940/#comment204637> this is not validate weight infos any more. - Vinod Kone On June 23, 2016, 6:04 a.m., zhou xing wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48940/ > ----------------------------------------------------------- > > (Updated June 23, 2016, 6:04 a.m.) > > > Review request for mesos, Anand Mazumdar, haosdent huang, and Vinod Kone. > > > Bugs: mesos-5496 > https://issues.apache.org/jira/browse/mesos-5496 > > > Repository: mesos > > > Description > ------- > > This patch refined the logic of updating weights in WeightsHandler. > Extracted the public code into new _updateWeights method so that > the new operator API can reuse this method to update weights. > > > Diffs > ----- > > src/master/master.hpp e983d1ba6ebcdaf2ace419201659e53edaa2a0aa > src/master/weights_handler.cpp 51c957f9ad278e60541b2bde3de2b6090a541ec4 > > Diff: https://reviews.apache.org/r/48940/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > zhou xing > >
