----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6665/ -----------------------------------------------------------
(Updated April 2, 2013, midnight) 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 04a8581 src/tests/allocator_zookeeper_tests.cpp cc6e22e src/tests/fault_tolerance_tests.cpp d02c9b2 src/tests/gc_tests.cpp 037fe09 src/tests/master_detector_tests.cpp fe3b91f src/tests/master_tests.cpp c74cb5e src/tests/resource_offers_tests.cpp 53b92ab src/tests/slave_recovery_tests.cpp b4276ed src/tests/sorter_tests.cpp 61e6038 src/tests/status_update_manager_tests.cpp a355ecd src/tests/utils.hpp 32784a7 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
