----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71750/ -----------------------------------------------------------
(Updated Nov. 22, 2019, 6:27 p.m.) Review request for mesos, Andrei Sekretenko, Greg Mann, and Meng Zhu. Changes ------- * Used default instead of all cases, per Andrei's suggestion, since it fits within our style guide recommendation: https://issues.apache.org/jira/browse/MESOS-3754 Bugs: MESOS-10026 https://issues.apache.org/jira/browse/MESOS-10026 Repository: mesos Description ------- This updates the handling to serialize directly to protobuf or json from the in-memory v0 state, bypassing expensive intermediate serialization / de-serialization / object construction / object destruction. This initial patch shows the approach that will be used for the other expensive calls. Note that this type of manual writing is more brittle and complex, but it can be mostly eliminated if we keep an up-to-date v1 GetState in memory in the future. When this approach is applied fully to GetState, it leads to the following improvement: Before: v0 '/state' response took 6.55 secs v1 'GetState' application/x-protobuf response took 24.08 secs v1 'GetState' application/json response took 22.76 secs After: v0 '/state' response took 8.00 secs v1 'GetState' application/x-protobuf response took 5.73 secs v1 'GetState' application/json response took 9.62 secs Diffs (updated) ----- src/master/http.cpp 1778664dddf19f9ab6d6c09ec35d64674ae488df src/master/master.hpp 8a140650a016c8afbfb39729eba2b5e78ea81c5f Diff: https://reviews.apache.org/r/71750/diff/3/ Changes: https://reviews.apache.org/r/71750/diff/2-3/ Testing ------- make check Thanks, Benjamin Mahler