You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Colin Williams <la...@gmail.com> on 2015/05/13 21:47:38 UTC

Review Request 34176: Removed allocator namespace

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

Review request for mesos.


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

Removed allocator namespace


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


Repository: mesos


Description (updated)
-------

As mentioned in the jira ticket, the allocator namespace should be removed to making the naming between mesos::master and mesos::slave consistent.


Diffs (updated)
-----

  include/mesos/master/allocator.hpp e821bf33536ab70cc2be45b9239c089e251f1f24 
  include/mesos/module/allocator.hpp 376eb4860322582f911d7a07253b79b1c9aa9292 
  src/examples/test_allocator_module.cpp 9679fd9f9a998834bc5329efa07d46a0403ab3f6 
  src/local/local.hpp 6bb25f16982d7be067b5f1307d5eca7745820e81 
  src/local/local.cpp dda25ab348c6430360c4b88e1d93dc70d14738d2 
  src/master/allocator/mesos/allocator.hpp 2681af5ff265b84c5174de3ad459292e006dcf97 
  src/master/allocator/mesos/hierarchical.hpp 09adced9d8712b3eeda885d598443791186890db 
  src/master/allocator/sorter/drf/sorter.hpp 4366710d6530b784aa5094813328d0e338239ba0 
  src/master/allocator/sorter/drf/sorter.cpp 2f69f384b95ff20d3ee429a4570a8cffa74d8e8b 
  src/master/allocator/sorter/sorter.hpp e2efb27b11dbea42dd73f81e5db0d6d2b0a6034b 
  src/master/main.cpp 18f8c3178459da0cbf23a1817ec49cd9d3998bfd 
  src/master/master.hpp af41216f4a8b81e58916c607a843965aeb4dfb5e 
  src/master/master.cpp ec32cd655c5a4fded0b682b320a6e0867e4e6468 
  src/tests/cluster.hpp 95061665974a7aca98c374735dd9520278d89043 
  src/tests/fault_tolerance_tests.cpp 3a27d82b2e24cc3821c846a0fe0458645b46a496 
  src/tests/hierarchical_allocator_tests.cpp 1a43dc72e739f3c55787716d680faa42a7d0d86f 
  src/tests/master_allocator_tests.cpp 534b2486a6a02ef32b5a7235d3ad30e82a78d6a5 
  src/tests/master_authorization_tests.cpp 5633c823c116cdb24fe2620c746a982e69ab91ca 
  src/tests/master_tests.cpp 75ffadae64ece4e3ff53abeefa5f6e8e3690d402 
  src/tests/mesos.hpp 563b833f8563ee6687ef95143815ed38dff71657 
  src/tests/mesos.cpp bc082e8d91deb2c5dd64bbc3f0a8a50fa7d19264 
  src/tests/partition_tests.cpp 1018e479a77a6b533f2dd392fedbdccb80e3ac1f 
  src/tests/rate_limiting_tests.cpp 49d907b0be45ccfed8af5c8fda89ad560e012c1e 
  src/tests/resource_offers_tests.cpp 882a9ff4d09aace486182828bf43b643b0d0c519 
  src/tests/slave_recovery_tests.cpp c036e9c992dd3377459caea543d3af3fd6279911 
  src/tests/sorter_tests.cpp 42442353afe7bd3d1a5b43992f8ae191ac19bdcd 

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


Testing (updated)
-------

Ran `make check`, everything passed.


Thanks,

Colin Williams