----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18752/#review36185 -----------------------------------------------------------
Looks like you're missing the diff Bernd! - Benjamin Hindman On March 4, 2014, 10:52 p.m., Bernd Mathiske wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18752/ > ----------------------------------------------------------- > > (Updated March 4, 2014, 10:52 p.m.) > > > Review request for mesos. > > > Bugs: MESOS-610 > https://issues.apache.org/jira/browse/MESOS-610 > > > Repository: mesos-git > > > Description > ------- > > MESOS-610: Split slave specific tests out of master_tests. > > Not that most of the tests in master_test.cpp are very master-specific to > begin with. > Nor are any of them terribly slave-specific. > Typically, any such tests concern the whole Mesos conglomerate including the > scheduler driver. > The submitted patch suggests a more or less plausible split, > but other divisions are easily thinkable. > > I'd also be happy with keeping them all in one file and renaming that file, > maybe to "mesos_tests.cpp"? > > > Diffs > ----- > > > Diff: https://reviews.apache.org/r/18752/diff/ > > > Testing > ------- > > Compiled and ran all tests ("make check") on Mac OS 10.9 / Clang 3.3 and > Ubuntu 13.10 / gcc 4.8.1. > > > Thanks, > > Bernd Mathiske > >
