----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49376/#review140620 -----------------------------------------------------------
src/master/allocator/sorter/drf/sorter.cpp (lines 272 - 275) <https://reviews.apache.org/r/49376/#comment206003> How about adjust the order a bit as following? CHECK(allocations[name].resources[slaveId].contains(resources)); const Resources resourcesQuantity = resources.createStrippedScalarQuantity(); CHECK(allocations[name].scalarQuantities.contains(resourcesQuantity)); - Guangya Liu On 七月 3, 2016, 8:36 a.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49376/ > ----------------------------------------------------------- > > (Updated 七月 3, 2016, 8:36 a.m.) > > > Review request for mesos, Alexander Rukletsov, Joris Van Remoortere, and > Michael Park. > > > Bugs: MESOS-5698 > https://issues.apache.org/jira/browse/MESOS-5698 > > > Repository: mesos > > > Description > ------- > > Added assertions to DRFSorter. > > > Diffs > ----- > > src/master/allocator/sorter/drf/sorter.cpp > 967290d4d1100208900b4b724422c3218abc23cb > > Diff: https://reviews.apache.org/r/49376/diff/ > > > Testing > ------- > > These assertions DO NOT PASS. They are conceptually correct, however -- after > r/49377 they pass on `make check`. > > > Thanks, > > Neil Conway > >
