----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68731/#review208679 -----------------------------------------------------------
PASS: Mesos patch 68731 was successfully built and tested. Reviews applied: `['68729', '68730', '68731']` All the build artifacts available at: http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2344/mesos-review-68731 - Mesos Reviewbot Windows On Sept. 16, 2018, 7:59 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68731/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2018, 7:59 p.m.) > > > Review request for mesos, Gastón Kleiman and Meng Zhu. > > > Bugs: MESOS-9239 > https://issues.apache.org/jira/browse/MESOS-9239 > > > Repository: mesos > > > Description > ------- > > This type replaces the use of hashmaps keyed by resource names in > favor of storing vectors of `pair<string,Value::Scalar>`, in order > to avoid the performance penalty of using hashmaps. > > Running *HierarchicalAllocator_BENCHMARK_Test.DeclineOffers/21 shows > the following improvement: > > Using 10000 agents and 1000 frameworks > Added 1000 frameworks in 42.49ms -> 42.85ms (no change) > Added 10000 agents in 7.69secs -> 4.89secs (normalized: 1 -> 0.64) > round 0 allocate() took 5.42secs -> 3.53secs (nomralized: 1 -> 0.65) > > > Diffs > ----- > > src/master/allocator/sorter/drf/sorter.hpp > 5a4fa5e2dca61168923261230b1f5c245354cbb7 > > > Diff: https://reviews.apache.org/r/68731/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Mahler > >