mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexander Rukletsov" <ruklet...@gmail.com>
Subject Re: Review Request 31775: Removed Master::Flags dependency from Allocator.
Date Wed, 01 Apr 2015 14:24:54 GMT

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

(Updated April 1, 2015, 2:24 p.m.)


Review request for mesos, Kapil Arya, Michael Park, and Niklas Nielsen.


Changes
-------

Addressed Niklas' comment.


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


Repository: mesos


Description
-------

On the way to moving allocator to public includes we need to remove the dependency to internal
Master::Flags type.


Diffs (updated)
-----

  src/master/allocator/allocator.hpp b67b8fddbd7a3fffc6fe24d5e77cd1db8cb6f69b 
  src/master/allocator/mesos/allocator.hpp fb898f1175b61b442204e6e38c69ccc2838a646f 
  src/master/allocator/mesos/hierarchical.hpp 9f9a631ee52a3ab194e678f9be139e8dabc7f3db 
  src/master/master.cpp 618db68ee4163b06e479cf3413eda4b63c9c5a4b 
  src/tests/hierarchical_allocator_tests.cpp 8861bf398e4bb17b0f74eab4f4af26202447ccef 
  src/tests/mesos.hpp 0e98572a62ae05437bd2bc800c370ad1a0c43751 

Diff: https://reviews.apache.org/r/31775/diff/


Testing
-------

make check (Mac OS 10.9.5, CentOS 7.0)


Thanks,

Alexander Rukletsov


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