-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49118/
-----------------------------------------------------------
(Updated June 24, 2016, 12:51 a.m.)
Review request for mesos, Joris Van Remoortere and Vinod Kone.
Changes
-------
Review comments: changelog and reformat a line.
Bugs: MESOS-5642
https://issues.apache.org/jira/browse/MESOS-5642
Repository: mesos
Description
-------
This is the second part of moving allocator to own package:
1. Move the non-versioned allocator.proto and allocator.hpp;
2. Move namespace of `Allocator` base class;
3. Fix allocation-module.md;
4. Call out allocator api change in docs/upgrades.md.
Diffs (updated)
-----
CHANGELOG e8cc37e4a96de023bfd0286c063ef6fcb0e0e292
docs/allocation-module.md 6cb1392aff9eeb5cb4a8f14c91df50f29d25948c
docs/upgrades.md 963b7d4797bb053d6bc1e60d3e8587465c2fc3c3
include/mesos/maintenance/maintenance.proto
458b42733fe7cb83dd43e5daf239929bea6fcb41
include/mesos/master/allocator.hpp a4743c5a31b18d96722a42d526bfb669d30e6e48
include/mesos/master/allocator.proto 8252abf47a19d5cc6c581c446b8756648fb2a054
include/mesos/module/allocator.hpp 5e65e69b6d62611f51e85aa9adaa283713896d4c
src/CMakeLists.txt d66186217c1319d4497640614ed4beee28602c38
src/Makefile.am 23cac95a805e46d216e9479fea09f2c1619c45a5
src/examples/test_allocator_module.cpp
bc28eb4146d4cfe1c3e04302eaa4f3f4ac91c4eb
src/local/local.hpp 0c86f97dc7c1a4ee7fe49c457de051eab03d5117
src/local/local.cpp b40ca7c0bc44561ba56baa3499e124f5a75a9489
src/master/allocator/allocator.cpp 8e76a3a33efd896a21960b0092dfdee800d211f5
src/master/allocator/mesos/allocator.hpp
64bce0fb143b109c26f923cd97d5facb393dee9d
src/master/allocator/mesos/hierarchical.hpp
c9cb1e114ad3407f86799c533fd6ae9c459f9062
src/master/allocator/mesos/hierarchical.cpp
5b2331b3f99252fd16f2514123006dd4ba7f1c0d
src/master/http.cpp 7daaf12a4086635bbc5aba5e3375c95e8899ac6e
src/master/main.cpp 01c827625927ed6b9d2dc6f2c4d8e7bd1b45677c
src/master/master.hpp fe57878dc59637459d5c5cdae0be2aa159133fa4
src/master/master.cpp 8def7156f4a05b39590321ce7743f7385a68bed0
src/tests/allocator.hpp 4081193abed29b99f60254e98ef9ebc981bde859
src/tests/cluster.hpp 3b37f963400eaea9f7489d182ce024b43f7db874
src/tests/cluster.cpp f25b887d8a5b805e61640630f6f26f905f6c4f03
src/tests/fault_tolerance_tests.cpp 5109b8310871c2b4efe209b3d6fcce1b803af788
src/tests/hierarchical_allocator_tests.cpp
6de0702f1960677abf9fd458576f7400a938cbb0
src/tests/master_allocator_tests.cpp 0425266abd70f52f1c6c4fc294bfbcfb0b0df4df
src/tests/master_authorization_tests.cpp
9ae6b9dc89538716d397ffd8826187e2bc6d7e8f
src/tests/master_maintenance_tests.cpp
b1043852a1618c5e2deafdc5437184ab593f1361
src/tests/master_slave_reconciliation_tests.cpp
f67bfd2ceec6303cfac5782417e824bfae7ca665
src/tests/master_tests.cpp ce9c192ce29ea6e1b3757f4b7459e7f92bb16389
src/tests/mesos.hpp b9de9b6dfe0e62df91d5b82006b799345b49fd3c
src/tests/mesos.cpp 15aaaa7979f227083580afc1349373f3e3364c65
src/tests/rate_limiting_tests.cpp 5d68ac170cb5a948df1429ffbf8e5b8f02f708d9
src/tests/reservation_tests.cpp 2cc352d230f5ba8ca15698b09a44377c6af53913
Diff: https://reviews.apache.org/r/49118/diff/
Testing
-------
`make && make check` on mac.
Thanks,
Zhitao Li
