> On Feb. 28, 2014, 5:24 p.m., Dominic Hamon wrote: > > src/log/recover.hpp, line 58 > > <https://reviews.apache.org/r/18600/diff/1/?file=506574#file506574line58> > > > > It looks like you always pass a value in for autoInitialization - is > > there a reason for it to have a default?
Yeah, because in tests, we use this function directly and don't pass a value by default. - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18600/#review35799 ----------------------------------------------------------- On Feb. 28, 2014, 12:59 a.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18600/ > ----------------------------------------------------------- > > (Updated Feb. 28, 2014, 12:59 a.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > Bugs: MESOS-984 > https://issues.apache.org/jira/browse/MESOS-984 > > > Repository: mesos-git > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/log/log.hpp 628a4b8 > src/log/log.cpp 7f855f2 > src/log/recover.hpp 634bc06 > src/log/recover.cpp 3403b47 > src/tests/log_tests.cpp 1e59b24 > > Diff: https://reviews.apache.org/r/18600/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
