You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Thomas Marshall <tw...@gmail.com> on 2013/05/30 19:48:59 UTC

Review Request: Add Slave and Framework struct to HierarchicalAllocatorProcess

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

Review request for mesos and Benjamin Hindman.


Description
-------

Cleans up the proliferation of hashmaps in HierarchicalAllocatorProcess.


Diffs
-----

  src/local/local.hpp 51324e2 
  src/local/local.cpp eb33e5d 
  src/master/allocator.hpp b68b67d 
  src/master/drf_sorter.hpp 79566cc 
  src/master/drf_sorter.cpp fe31a00 
  src/master/hierarchical_allocator_process.hpp ebd97e4 
  src/master/main.cpp 06c265a 
  src/master/master.hpp 38d2a51 
  src/master/master.cpp 5ee7763 
  src/master/sorter.hpp 73db6b1 
  src/tests/allocator_tests.cpp 52e2b03 
  src/tests/allocator_zookeeper_tests.cpp 1034d72 
  src/tests/cluster.hpp 25cd554 
  src/tests/mesos.hpp 12298ae 
  src/tests/mesos.cpp 4ddf032 
  src/tests/resource_offers_tests.cpp b066403 
  src/tests/sorter_tests.cpp 95e8d28 
  third_party/libprocess/third_party/stout/include/stout/stringify.hpp 136316d 

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


Testing
-------


Thanks,

Thomas Marshall


Re: Review Request: Add Slave and Framework struct to HierarchicalAllocatorProcess

Posted by Benjamin Hindman <be...@berkeley.edu>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11541/#review21207
-----------------------------------------------------------

Ship it!


Ship It!

- Benjamin Hindman


On May 30, 2013, 6:13 p.m., Thomas Marshall wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11541/
> -----------------------------------------------------------
> 
> (Updated May 30, 2013, 6:13 p.m.)
> 
> 
> Review request for mesos and Benjamin Hindman.
> 
> 
> Description
> -------
> 
> Cleans up the proliferation of hashmaps in HierarchicalAllocatorProcess.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/stout/include/stout/stringify.hpp 136316d 
>   src/local/local.hpp a165265 
>   src/local/local.cpp c35c19a 
>   src/master/allocator.hpp b68b67d 
>   src/master/drf_sorter.hpp 79566cc 
>   src/master/drf_sorter.cpp fe31a00 
>   src/master/hierarchical_allocator_process.hpp 28a7879 
>   src/master/main.cpp 3c1cf4c 
>   src/master/master.hpp 82e3596 
>   src/master/master.cpp d031b95 
>   src/master/sorter.hpp 73db6b1 
>   src/tests/allocator_tests.cpp b153dee 
>   src/tests/allocator_zookeeper_tests.cpp 1034d72 
>   src/tests/cluster.hpp 25cd554 
>   src/tests/mesos.hpp 12298ae 
>   src/tests/mesos.cpp d811267 
>   src/tests/resource_offers_tests.cpp b066403 
>   src/tests/sorter_tests.cpp 95e8d28 
> 
> Diff: https://reviews.apache.org/r/11541/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Thomas Marshall
> 
>


Re: Review Request: Add Slave and Framework struct to HierarchicalAllocatorProcess

Posted by Thomas Marshall <tw...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11541/
-----------------------------------------------------------

(Updated May 30, 2013, 6:13 p.m.)


Review request for mesos and Benjamin Hindman.


Changes
-------

Rebased.


Description
-------

Cleans up the proliferation of hashmaps in HierarchicalAllocatorProcess.


Diffs (updated)
-----

  3rdparty/libprocess/3rdparty/stout/include/stout/stringify.hpp 136316d 
  src/local/local.hpp a165265 
  src/local/local.cpp c35c19a 
  src/master/allocator.hpp b68b67d 
  src/master/drf_sorter.hpp 79566cc 
  src/master/drf_sorter.cpp fe31a00 
  src/master/hierarchical_allocator_process.hpp 28a7879 
  src/master/main.cpp 3c1cf4c 
  src/master/master.hpp 82e3596 
  src/master/master.cpp d031b95 
  src/master/sorter.hpp 73db6b1 
  src/tests/allocator_tests.cpp b153dee 
  src/tests/allocator_zookeeper_tests.cpp 1034d72 
  src/tests/cluster.hpp 25cd554 
  src/tests/mesos.hpp 12298ae 
  src/tests/mesos.cpp d811267 
  src/tests/resource_offers_tests.cpp b066403 
  src/tests/sorter_tests.cpp 95e8d28 

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


Testing
-------


Thanks,

Thomas Marshall