> On Jan. 18, 2017, 7:24 a.m., Greg Mann wrote: > > src/tests/slave_recovery_tests.cpp, line 1204 > > <https://reviews.apache.org/r/55612/diff/2/?file=1606525#file1606525line1204> > > > > As long as you're cleaning this up, might as well fix the spacing - > > should be just one space before the comment begins. > > Vinod Kone wrote: > Do you mean the spacing between "Return());" and "//" ? Quite a few of > such comments in this file use more than one space for this case.
OK, I was just referencing our style guide which specifies a single space for trailing comments. - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55612/#review162031 ----------------------------------------------------------- On Jan. 17, 2017, 1:06 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55612/ > ----------------------------------------------------------- > > (Updated Jan. 17, 2017, 1:06 p.m.) > > > Review request for mesos and Greg Mann. > > > Bugs: MESOS-6922 > https://issues.apache.org/jira/browse/MESOS-6922 > > > Repository: mesos > > > Description > ------- > > Just like RecoverHTTPTerminatedExecutor test, the scheduler should > handle duplicate status updates after agent restarts. See the > attached ticket for details. > > > Diffs > ----- > > src/tests/slave_recovery_tests.cpp 9323cbb74dd555fe6c44af1ef6da70e0c2d76dcb > > Diff: https://reviews.apache.org/r/55612/diff/ > > > Testing > ------- > > make check > > Ran in a loop for 20 times. > > > Thanks, > > Vinod Kone > >