-----------------------------------------------------------
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

Reply via email to