mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jie Yu <yujie....@gmail.com>
Subject Review Request 63523: Updated `Allocator::updateAllocation` to take `ResourceConversion`s.
Date Thu, 02 Nov 2017 19:36:19 GMT

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

Review request for mesos, Benjamin Bannier, Benjamin Mahler, Gaston Kleiman, Greg Mann, Jan
Schlicht, Vinod Kone, and Jiang Yan Xu.


Repository: mesos


Description
-------

Instead of taking offer operations, it's more clear to make allocator
not aware of offer operations. This patch changes the interface of the
`updateAllocation` call to take `ResourceConversion`s.


Diffs
-----

  include/mesos/allocator/allocator.hpp 537658bf4e68ac7f271248af0e1e35df2b19aef8 
  src/master/allocator/mesos/allocator.hpp 903edf68e812e022cddedd40a2cb51726b2a181b 
  src/master/allocator/mesos/hierarchical.hpp c2346054b2c98516f15ab8ce2dc798224ff4def4 
  src/master/allocator/mesos/hierarchical.cpp 848e9da1b61f411449c0cd2409e98ec733ac10ee 
  src/master/master.cpp e047462018af50b9047b8939e922d80c4771fb28 
  src/tests/allocator.hpp f1c0d148886b7b2936918b7d7c77c03ee766ee4a 
  src/tests/hierarchical_allocator_tests.cpp 48b48ad70f44cc2232c2a29699267027f9937b8a 


Diff: https://reviews.apache.org/r/63523/diff/1/


Testing
-------

make check


Thanks,

Jie Yu


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