mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vinod Kone" <vinodk...@gmail.com>
Subject Re: Review Request: Refactored Allocator
Date Thu, 25 Oct 2012 01:58:38 GMT

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

(Updated Oct. 25, 2012, 1:58 a.m.)


Review request for mesos, Benjamin Hindman, Thomas Marshall, and Ben Mahler.


Changes
-------

It works!


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

Refactored Allocator


Description (updated)
-------

Specifically:

--> Allocator writers don't need to do extra work to make their allocator async. This is
guaranteed/enforced by the Allocator interface.

--> Inside master, the allocator calls looks like method calls instead of dispatches.


Diffs (updated)
-----

  src/local/local.hpp cd0483c72f919e08a67b4eeaebef3b62b0ff9798 
  src/local/local.cpp 67c7a7c260d2ea610fb64b491f9bf7a22ec53aac 
  src/master/allocator.hpp 9fd790564d1361b9bdb2111f29348a631f66c831 
  src/master/allocator.cpp 3f822912a4e0c50c1612eac98ae85447aae023d4 
  src/master/drf_sorter.hpp 9c43ba32c6ed04fd6960524b5925c5c27c3e4d2d 
  src/master/drf_sorter.cpp 0ab4a14ca0bfb721de00e29d993fd4aa08a27ac5 
  src/master/hierarchical_allocator_process.hpp 266d339bdb72f8c63288d91c1f514da07ee9acf2 
  src/master/main.cpp 813ef7de8e57b1cbd688e6e34e360c7b191ed641 
  src/master/master.hpp 146af017bbb6da9bd44acb53a4f1ee0ffbedd64f 
  src/master/master.cpp 82e4dc704e5c67ec178bd058934896328308d868 
  src/master/sorter.hpp 80d0893b1f988898e40ae3a2f6f19a117ad5f72d 
  src/sched/sched.cpp 5a4e594a3a8a2089cf74e7e418f2e7bde4c03e9d 
  src/tests/allocator_tests.cpp ed52ff61b1820cb3f5f7f2ae421fb3f84a85939a 
  src/tests/allocator_zookeeper_tests.cpp bd408c22df5edc3684791e509ce1a4c210553922 
  src/tests/fault_tolerance_tests.cpp 558bf41ab6f4ba319ec3b23f3347f98e235260f9 
  src/tests/gc_tests.cpp 1de240579a1cc6af8738d2ffb5da1b3e01c4b19e 
  src/tests/master_detector_tests.cpp fb32af8130d3f359eecba5389c2e42c247d6b70e 
  src/tests/master_tests.cpp 37e97e0a58a0128579f155183b7e769ab4e1b452 
  src/tests/resource_offers_tests.cpp c50da38508694f956ad104fb160896eb9b9d5fea 
  src/tests/utils.hpp 4066c02b23db15310104955465c4e4afd7bcad5b 

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


Testing (updated)
-------

make check


Thanks,

Vinod Kone


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