----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57527/ -----------------------------------------------------------
(Updated March 20, 2017, 9:18 p.m.) Review request for mesos, Benjamin Bannier, Benjamin Mahler, and Michael Park. Changes ------- Tweak comment. Repository: mesos Description ------- Previously, DRFSorter only kept track of weights for clients currently in the sorter; the allocator supplied the current weight when adding a client to the sorter. This commit changes the sorter to keep track of all weights, not just those for the active clients in the sorter. The allocator can now just pass along role weights to the role sorters, rather than needing to track them itself. This commit changes the sorter API. Diffs (updated) ----- src/master/allocator/mesos/hierarchical.hpp f84b0574ce9a392c9528c87b04b01dbb2053cff7 src/master/allocator/mesos/hierarchical.cpp 8d54a8cca1bb478f4437f68c5e14f66a9f9bb9e9 src/master/allocator/sorter/drf/sorter.hpp 76329220e1115c1de7810fb69b943c78c078be59 src/master/allocator/sorter/drf/sorter.cpp ed54680cecb637931fc344fbcf8fd3b14cc24295 src/master/allocator/sorter/sorter.hpp b3029fcf7342406955760da53f1ae736769f308c src/tests/sorter_tests.cpp ec0636beb936d46a253d19322f2157abe95156b6 Diff: https://reviews.apache.org/r/57527/diff/5/ Changes: https://reviews.apache.org/r/57527/diff/4-5/ Testing ------- `make check` Thanks, Neil Conway