mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Neil Conway <>
Subject Re: Review Request 49375: Simplified DRFSorter to not track per-slave total resources.
Date Mon, 04 Jul 2016 09:01:13 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated July 4, 2016, 9:01 a.m.)

Review request for mesos, Alexander Rukletsov, Joris Van Remoortere, and Michael Park.


Tweak comment per review.

Bugs: MESOS-5698

Repository: mesos


The DRFSorter previously kept the total resources at each slave, along
with the total quantity of resources in the cluster. The latter figure
is what most of the clients of the sorter care about. In the few places
where the previous coding was using the per-slave total resources, it is
relatively easy to adjust the code to remove this dependency.

As part of this change, remove `total()` and `update(const SlaveID&
slaveId, const Resources& resources)` from the Sorter interface. The
former was unused; for the latter, code that used it can instead be
rewritten to adjust the total resources in the cluster by first removing
the previous resources at a slave and then adding the new resources.

Diffs (updated)

  src/master/allocator/mesos/hierarchical.cpp eca949e07abb00423a2f35a56eedc5d4287d81f3 
  src/master/allocator/sorter/drf/sorter.hpp 0aa1a71da4501a3b469d07538a043b4c1d74d688 
  src/master/allocator/sorter/drf/sorter.cpp 967290d4d1100208900b4b724422c3218abc23cb 
  src/master/allocator/sorter/sorter.hpp 5ce6bb82b0127257d97daf0cea6d1d0db405bf83 
  src/tests/sorter_tests.cpp 6fc58829892dc0223140f1b47593a3e5853cace5 



make check on OSX and recent Arch Linux.


Neil Conway

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message