mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Meng Zhu <>
Subject Review Request 71515: Fixed a bug where sorter may leak clients.
Date Thu, 19 Sep 2019 03:25:18 GMT

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

Review request for mesos, Andrei Sekretenko and Benjamin Mahler.

Bugs: MESOS-9975

Repository: mesos


It is possible to update allocations to empty in the sorter.
See MESOS-9015. When this happens, sorter will leak client
entries, since it does not erase the corresponding entries.

This patch fixes this issue. Also added a regression test.


  src/master/allocator/mesos/sorter/drf/sorter.hpp 2a861e2ea5fb61d20e50efbf3d94ee9757bca592

  src/master/allocator/mesos/sorter/random/sorter.hpp bc5580959046492cbf6c38fe2cd7a7d9e11c99f4

  src/tests/sorter_tests.cpp 64627c4ea57af19eb00bdff598936e23b55541f5 



make check
Added test fail on master, pass with the patch


Meng Zhu

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