----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35816/ -----------------------------------------------------------
Review request for mesos, Ben Mahler and Jie Yu. Repository: mesos Description ------- Fixed the incorrect `CHECK_EQ` introduced in [r35699](https://reviews.apache.org/r/35699/). >From @bmahler: > Whoops, Jie just noticed that this isn't correct because 'updatedAllocation' > is for the framework only, whereas 'total' and 'available' are for all > frameworks on the slave. Diffs ----- src/master/allocator/mesos/hierarchical.hpp ef18ff850addfb5ce3500ed28e8ffd801e2d24eb src/tests/hierarchical_allocator_tests.cpp 3258840135290cd008ca09235d18b7f093dafd2e Diff: https://reviews.apache.org/r/35816/diff/ Testing ------- (1) Added a test `HierarchicalAllocatorTest.UpdateAllocationMultipleFrameworks` which breaks the previous `CHECK_EQ` condition. (2) `make check` Thanks, Michael Park
