> On June 11, 2016, 8:33 p.m., Anand Mazumdar wrote: > > src/internal/evolve.hpp, lines 120-121 > > <https://reviews.apache.org/r/48585/diff/2/?file=1415685#file1415685line120> > > > > It's fine to evolve `JSON::Object` directly into `v1` to avoid making > > an extra copy. We adopted a similar approach with the Scheduler/Executor > > API's i.e. we had an `evolve` from old style messages to `v1::Call`.
Thanks a lot for your review! It solve all my concerns about this patch. I just updated, may you help review this again, thank you in advance. - haosdent ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48585/#review137137 ----------------------------------------------------------- On June 12, 2016, 4:15 a.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48585/ > ----------------------------------------------------------- > > (Updated June 12, 2016, 4:15 a.m.) > > > Review request for mesos, Anand Mazumdar, zhou xing, Jay Guo, and Vinod Kone. > > > Bugs: MESOS-5593 > https://issues.apache.org/jira/browse/MESOS-5593 > > > Repository: mesos > > > Description > ------- > > Devolved v1 operator protos to unversioned operator protos in Master. > > > Diffs > ----- > > src/internal/devolve.hpp 00842bb0de1dd587f2b47c79f17c0e7bd7f51189 > src/internal/devolve.cpp 4233246c6838f85189f1b4c7e66d2bc0a3bf5408 > src/internal/evolve.hpp 66a3deaa94939ad2233d944ba35ac7e5cbe682e7 > src/internal/evolve.cpp 7f16cbda7da6c838648cca909368973e7298730b > src/master/http.cpp 6e1bf9557a854a89fa9173223295816a9e114e7c > src/master/master.hpp 2c45dab291a153b42809ab12e4252bf58559feeb > src/master/validation.hpp e1271bbaebaac52a078eedfdc2299c7c6f4474f8 > src/master/validation.cpp 7b9c2281b2ab1295211f0dd385cd77947fbd63be > > Diff: https://reviews.apache.org/r/48585/diff/ > > > Testing > ------- > > > Thanks, > > haosdent huang > >
