----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53313/#review154320 -----------------------------------------------------------
src/local/local.cpp (lines 214 - 228) <https://reviews.apache.org/r/53313/#comment223818> If you move `#ifdef` up one, you won't need to add the extra `EXIT(..)`. src/master/main.cpp (line 37) <https://reviews.apache.org/r/53313/#comment223817> Oops, you probably meant an `#ifdef` here :) src/master/main.cpp (lines 392 - 398) <https://reviews.apache.org/r/53313/#comment223819> Similar `#ifdef`-ing shifting here - Joseph Wu On Oct. 31, 2016, 12:31 p.m., Alex Clemmer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53313/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2016, 12:31 p.m.) > > > Review request for mesos, Daniel Pravat and Joseph Wu. > > > Repository: mesos > > > Description > ------- > > Many agent tests currently use the master in some form. Of those, > relatively few use the leveldb-based persistence. Since we cannot > currently compile leveldb on Windows, and since running these tests on > Windows involves compiling and running the master, this commit will > remove the direct dependency on the replicated log for windows builds of > the master. > > > Diffs > ----- > > src/local/local.cpp 257179443827ffdfc946ed655a6840fcea70d454 > src/master/main.cpp 2d2dfb7d632f3c7be1796efd8f0a1f4d18760261 > > Diff: https://reviews.apache.org/r/53313/diff/ > > > Testing > ------- > > > Thanks, > > Alex Clemmer > >
