> On July 10, 2018, 5:24 a.m., Michael Park wrote: > > 3rdparty/stout/include/stout/json.hpp > > Line 855 (original), 892 (patched) > > <https://reviews.apache.org/r/67861/diff/1/?file=2056838#file2056838line892> > > > > Perhaps leave a small comment + a pointer to > > https://github.com/kazuho/picojson/blob/v1.3.0/picojson.h#L820-L870 here.
Will do. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67861/#review205891 ----------------------------------------------------------- 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 > >
