----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6665/ -----------------------------------------------------------
(Updated March 21, 2013, 11:10 p.m.) Review request for mesos and Benjamin Hindman. Changes ------- Updated to trunk. Description ------- There has been an unfortunate trend in the HierarchicalAllocatorProcess code to use lots of hashmaps between slaveIds or frameworkIds and some attribute of those frameworks/slaves. This patch collapses those into two hashmaps, one for slaves and one for frameworks, mapping ids to structs containing the appropriate attributes. The patch also introduces an allocator namespace. Diffs (updated) ----- src/local/local.hpp 2633d25 src/local/local.cpp f43ab2a src/master/allocator.hpp b68b67d src/master/drf_sorter.hpp 79566cc src/master/drf_sorter.cpp fe31a00 src/master/hierarchical_allocator_process.hpp 2422fbd src/master/main.cpp d6e1c73 src/master/master.hpp 9776a7c src/master/master.cpp 5b0e8c0 src/master/sorter.hpp 73db6b1 src/tests/allocator_tests.cpp e8b835f src/tests/allocator_zookeeper_tests.cpp 802a508 src/tests/fault_tolerance_tests.cpp 9d3f8b1 src/tests/gc_tests.cpp 67bf957 src/tests/master_detector_tests.cpp fe3b91f src/tests/master_tests.cpp 16f7637 src/tests/resource_offers_tests.cpp 53b92ab src/tests/slave_recovery_tests.cpp 47f9b0f src/tests/sorter_tests.cpp 61e6038 src/tests/status_update_manager_tests.cpp 9ce47b5 src/tests/utils.hpp 5020357 third_party/libprocess/third_party/stout/include/stout/stringify.hpp 136316d Diff: https://reviews.apache.org/r/6665/diff/ Testing ------- make check Thanks, Thomas Marshall
