----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67890/#review206008 -----------------------------------------------------------
Patch looks great! Reviews applied: [67877, 67889, 67890] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On July 11, 2018, 10:53 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67890/ > ----------------------------------------------------------- > > (Updated July 11, 2018, 10:53 p.m.) > > > Review request for mesos and Greg Mann. > > > Repository: mesos > > > Description > ------- > > When adding / removing metrics, we were performing double lookups > on the metrics map. Removing these provides a slight performance > improvement. > > > Diffs > ----- > > 3rdparty/libprocess/src/metrics/metrics.cpp > 4883c9acaa0cc568e27944661a8208f7b2a356a1 > > > Diff: https://reviews.apache.org/r/67890/diff/1/ > > > Testing > ------- > > Before: > ``` > Added 100000 counters in 2.455274623secs > Snapshot of 100000 counters in 809.816666ms > HTTP /snapshot of 100000 counters in 1.739941138secs > Removed 100000 counters in 3.31390385secs > ``` > > After: > ``` > Added 100000 counters in 1.954696011secs > Snapshot of 100000 counters in 826.540283ms > HTTP /snapshot of 100000 counters in 1.775644115secs > Removed 100000 counters in 2.404567921secs > ``` > > > Thanks, > > Benjamin Mahler > >
