----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71978/#review219252 -----------------------------------------------------------
Fix it, then Ship it! Can you also adjust the commit message to indicate that the jsonify/serialize methods in fact were just moved? IMO, this can make comprehending git blame a bit simpler in some cases. src/master/master.hpp Lines 1388-1417 (patched) <https://reviews.apache.org/r/71978/#comment307415> Consider making all those `jsonify/serialize` methods (except for `*getState` private). This isn't critical as this class holds almost no state, but they are not really intended for external use, right? src/master/readonly_handler.cpp Lines 1669 (patched) <https://reviews.apache.org/r/71978/#comment307412> That's 81 characters - did this pass through the pre-commit hooks? src/master/readonly_handler.cpp Lines 2312 (patched) <https://reviews.apache.org/r/71978/#comment307413> redundant blank line - Andrei Sekretenko On Jan. 9, 2020, 5:48 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71978/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2020, 5:48 p.m.) > > > Review request for mesos, Andrei Sekretenko and Greg Mann. > > > Bugs: MESOS-9497 > https://issues.apache.org/jira/browse/MESOS-9497 > > > Repository: mesos > > > Description > ------- > > Using the same approach taken for the v0 read-only endpoints, > this enables parallel reads for the GET_* v1 master calls. > > > Diffs > ----- > > src/master/http.cpp 16dbed3aad7aeac7f45eb44ffc98cb9df6546aff > src/master/master.hpp c9d417c23981a3908b419ce70cdf9a0096e37c17 > src/master/readonly_handler.cpp 500ce3f6a0f3afc3193586020fa5c253e4c6801c > src/tests/master_load_tests.cpp ac16fe744ec2a8d4b4af005b91a5560f0b2f3ef3 > > > Diff: https://reviews.apache.org/r/71978/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Mahler > >
