----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43879/ -----------------------------------------------------------
(Updated March 15, 2016, 3:51 p.m.) Review request for mesos, Alexander Rukletsov and Ben Mahler. Changes ------- Addressed offline comments from Ben Mahler. Bugs: MESOS-4718 https://issues.apache.org/jira/browse/MESOS-4718 Repository: mesos Description ------- Added allocator metrics for number of allocations made. Diffs (updated) ----- docs/monitoring.md 827f7073204fcf8575ca980a5571c8be4f5e4110 src/master/allocator/mesos/hierarchical.cpp 70291075c00a9a557529c2562dedcfc6c6c3ec32 src/master/allocator/mesos/metrics.hpp d04e9a11c77b6fb2d522608e3085f81f8a6657ca src/master/allocator/mesos/metrics.cpp 46dd7f7b3d8d49c59b42e789fe5dcd49f1693ace src/tests/hierarchical_allocator_tests.cpp 459e02576f6d05abbbcc83ae5cabac5c66e93f05 Diff: https://reviews.apache.org/r/43879/diff/ Testing ------- make check (OS X) I confirmed that this does not lead to general performance regressions in the allocator; this is partially expected since the added code only inserts metrics in the allocator while the actual work is perform asynchronously. These tests where performed with `HierarchicalAllocator_BENCHMARK_Test.DeclineOffers` on an optimized build under OS X using clang(trunk) as compiler. Thanks, Benjamin Bannier