----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67861/#review205887 -----------------------------------------------------------
PASS: Mesos patch 67861 was successfully built and tested. Reviews applied: `['67861']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/1890/mesos-review-67861 - Mesos Reviewbot Windows On July 10, 2018, 1:40 a.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67861/ > ----------------------------------------------------------- > > (Updated July 10, 2018, 1:40 a.m.) > > > Review request for mesos, Greg Mann and Michael Park. > > > Repository: mesos > > > Description > ------- > > When looking at some profiling data, it looks as though we approximately > double the cost of JSON parsing due to having to convert from > `picojson::value` to `JSON::Value`. > > Michael Park pointed me to the parsing "context" that's customizable in > picojson. This patch replaces our conversion with a parsing context in > order to parse directly into JSON::Value and avoid copying. > > > Diffs > ----- > > 3rdparty/stout/include/stout/json.hpp > c374e29829e4ea02ab3a8494b6ab28c166579516 > > > Diff: https://reviews.apache.org/r/67861/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Mahler > >
