> On July 10, 2018, 5:41 a.m., Michael Park wrote: > > 3rdparty/stout/include/stout/json.hpp > > Lines 343-347 (original), 352-377 (patched) > > <https://reviews.apache.org/r/67861/diff/1/?file=2056838#file2056838line352> > > > > While we're here, we could also replace these with the pointer versions > > of `boost::get`. We might get some better performance: > > https://godbolt.org/g/B84cLb
Huh.. that's pretty suprising to me, is it surprising to you? I can look into that soon to see if it makes a difference. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67861/#review205893 ----------------------------------------------------------- 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 > >
