----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35664/ -----------------------------------------------------------
(Updated June 19, 2015, 10:21 p.m.) Review request for mesos, Ben Mahler and Vinod Kone. Changes ------- Review comments. Added a test. Bugs: MESOS-2891 https://issues.apache.org/jira/browse/MESOS-2891 Repository: mesos Description ------- Improved the performance of DRF sorter by caching the scalars. Diffs (updated) ----- src/master/allocator/sorter/drf/sorter.hpp 35dc1a4d0b5e61b26a07c2c53583d75896aff27c src/master/allocator/sorter/drf/sorter.cpp c5f4caf68aff26a9d4809eff11df98d0137aa401 src/tests/sorter_tests.cpp 435e0bfeb28c1a9ea124312a8b97a869945ac87f Diff: https://reviews.apache.org/r/35664/diff/ Testing ------- make check With this patch: [==========] Running 6 tests from 1 test case. [----------] Global test environment set-up. [----------] 6 tests from SlaveCount/HierarchicalAllocator_BENCHMARK_Test [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/0 Added 1000 slaves in 668.950475ms [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/0 (844 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/1 Added 5000 slaves in 3.335592974secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/1 (4244 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/2 Added 10000 slaves in 6.675837646secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/2 (8527 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/3 Added 20000 slaves in 13.411382999secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/3 (17177 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/4 Added 30000 slaves in 20.012000768secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/4 (25567 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/5 Added 50000 slaves in 33.377255617secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/5 (42960 ms) [----------] 6 tests from SlaveCount/HierarchicalAllocator_BENCHMARK_Test (99346 ms total) [----------] Global test environment tear-down [==========] 6 tests from 1 test case ran. (99452 ms total) [ PASSED ] 6 tests. Baseline (pre MESOS-2373) [==========] Running 6 tests from 1 test case. [----------] Global test environment set-up. [----------] 6 tests from SlaveCount/HierarchicalAllocator_BENCHMARK_Test [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/0 Added 1000 slaves in 492.832376ms [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/0 (602 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/1 Added 5000 slaves in 2.432903553secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/1 (2991 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/2 Added 10000 slaves in 4.866961208secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/2 (6014 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/3 Added 20000 slaves in 9.736516799secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/3 (12067 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/4 Added 30000 slaves in 14.599861839secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/4 (18011 ms) [ RUN ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/5 Added 50000 slaves in 24.30015991secs [ OK ] SlaveCount/HierarchicalAllocator_BENCHMARK_Test.AddSlave/5 (30097 ms) [----------] 6 tests from SlaveCount/HierarchicalAllocator_BENCHMARK_Test (69784 ms total) [----------] Global test environment tear-down [==========] 6 tests from 1 test case ran. (69820 ms total) [ PASSED ] 6 tests. Thanks, Jie Yu