----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28061/#review61584 -----------------------------------------------------------
Patch looks great! Reviews applied: [27496, 27497, 27498, 27499, 27500, 27501, 27502, 27503, 27504, 27505, 27506, 27507, 27508, 27509, 27510, 28061] All tests passed. - Mesos ReviewBot On Nov. 14, 2014, 11:58 p.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28061/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2014, 11:58 p.m.) > > > Review request for mesos and Niklas Nielsen. > > > Repository: mesos-git > > > Description > ------- > > We should really be including <process/timer.hpp> but there are currently too > many circular dependencies in header files that would need to get moved to > translation units first. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/clock.hpp > eb157cac996c48fedc69002b9d80adaba1ed85d1 > 3rdparty/libprocess/include/process/timeout.hpp > 0bf63e11a7a63b714aafb5386bf2d169260396ce > > Diff: https://reviews.apache.org/r/28061/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
