-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70344/
-----------------------------------------------------------

(Updated March 29, 2019, 2:28 p.m.)


Review request for mesos and Benjamin Mahler.


Changes
-------

Addressed Ben's comment


Summary (updated)
-----------------

Fixed a bug where disk quota may be under allocated.


Bugs: MESOS-9692
    https://issues.apache.org/jira/browse/MESOS-9692


Repository: mesos


Description (updated)
-------

The resources chopping logic in the allocator
currently overly shrinks resources with the same
name (e.g. vanilla disk and mount disk) if there
are multiple such resources. Currently only
different disk resources might share the same
name (we only chop unreserved/non-revocable/non-shared
resources). This would lead to disk quota being under
allocated. This patch corrects the shrinking logic.
See MESOS-9692.


Diffs (updated)
-----

  src/master/allocator/mesos/hierarchical.cpp 
8bc749903b8ceb09a02e260919377483479302b5 
  src/tests/hierarchical_allocator_tests.cpp 
a34fa96fbc455d830c1fd7c1df83f3db72c96ee3 


Diff: https://reviews.apache.org/r/70344/diff/2/

Changes: https://reviews.apache.org/r/70344/diff/1-2/


Testing
-------

make check
Added a dedicated test. The test fails without the fix and passes with the fix.


Thanks,

Meng Zhu

Reply via email to