----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38519/#review99655 -----------------------------------------------------------
Patch looks great! Reviews applied: [38514, 38516, 38519] All tests passed. - Mesos ReviewBot On Sept. 19, 2015, 1:26 a.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38519/ > ----------------------------------------------------------- > > (Updated Sept. 19, 2015, 1:26 a.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-3037 > https://issues.apache.org/jira/browse/MESOS-3037 > > > Repository: mesos > > > Description > ------- > > The reason that I was not using suppress() is because suppress is > already defined in > 3rdparty/libprocess/3rdparty/stout/include/stout/os/signals.hpp > > > Diffs > ----- > > src/master/http.cpp 7cca8b1d3118fe99de79cfaf5360825012d830da > src/master/master.hpp 6805177d7c03b701708e1d217a7bc0ac7c79889e > src/master/master.cpp 64e5fb9e27b2e6797b2ce87c0e1a3ef8a2943e27 > > Diff: https://reviews.apache.org/r/38519/diff/ > > > Testing > ------- > > Platform: Ubuntu:14.04 > make > make check > > > Thanks, > > Guangya Liu > >
