----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44964/ -----------------------------------------------------------
(Updated March 29, 2016, 12:25 a.m.) Review request for mesos and Jie Yu. Changes ------- Remove a few more unnecessary includes. Repository: mesos Description ------- This mostly targets unnecessary inclusion of stout headers, which can be quite expensive to compile. Diffs (updated) ----- src/credentials/credentials.hpp 32492f2fa4126d26afe3ab0b523891023deb6a67 src/examples/persistent_volume_framework.cpp 3848e247994b511fb2b1dcde337f8f37bb7472da src/master/contender.cpp 9ad49ce10439fb41d78d52eaa4c1e6b9c5c7f735 src/slave/state.cpp 94b6959cbca7ba14dfc70caa3402fe7d72d4757a src/tests/fetcher_tests.cpp bd86c962b37e2c584e261ba95798ac74a07091c2 src/tests/group_tests.cpp ed10f1fe8a5cd2a353175e13ec3c464da69956f4 src/tests/master_contender_detector_tests.cpp bbce379e5a0a0ca608579d0ab2b10970e9cd5ef1 src/tests/master_quota_tests.cpp c2b46d23002481e63ff162e8628f9b974e3e8ef9 src/tests/mesos.hpp f2a29d0ddce5ef2b9ef2f0155c428311a67dbab3 src/tests/module_tests.cpp eddb0e5484964595797758e2742b10431e76c039 src/tests/paths_tests.cpp 7952ae239470580f06df373c80f9115634b42640 src/tests/persistent_volume_tests.cpp aa097c3da1237e564dd4081c23034ab185be09bd src/tests/script.cpp 36ddf40fc003435f852ca138f66c4f33def1637b src/tests/state_tests.cpp 8fb2798e12fe77d6b93268e71e3a8261cb0842d5 src/tests/status_update_manager_tests.cpp 801767ba6b4a08a4f653a7735bf3a581a0f8f95f src/tests/zookeeper_test_server.hpp e2d8218fd7000904acaee2221da648c665da8dfc src/zookeeper/group.cpp 7aa95a7b7eafe0a2c49dc077b3a4abd688e5d8e8 Diff: https://reviews.apache.org/r/44964/diff/ Testing ------- `make check` Timed the compilation of a few files to verify the compilation time improvement -- e.g., removing an unused include of `stout/os.hpp` reduces compile time by ~1 second on my laptop. Thanks, Neil Conway
