> On Sept. 6, 2016, 2:14 p.m., Vinod Kone wrote: > > src/tests/partition_tests.cpp, line 723 > > <https://reviews.apache.org/r/50706/diff/2/?file=1487823#file1487823line723> > > > > `SlaveObserver`
Fixed the other places where this occurs as well. > On Sept. 6, 2016, 2:14 p.m., Vinod Kone wrote: > > src/tests/partition_tests.cpp, line 710 > > <https://reviews.apache.org/r/50706/diff/2/?file=1487823#file1487823line710> > > > > hmm. didn't realize that the TODO in `Master::reconcile` regarding > > assumption is actually wrong! tasks could be unknown to the master but > > known to the agent. not sure why we made that assumption. I believe that the comment in `Master::reconcile` assumes that the master hasn't failed over (see the `NOTE` at the top of the function). - Neil ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50706/#review147827 ----------------------------------------------------------- On Aug. 29, 2016, 9:49 a.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50706/ > ----------------------------------------------------------- > > (Updated Aug. 29, 2016, 9:49 a.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-4049 > https://issues.apache.org/jira/browse/MESOS-4049 > > > Repository: mesos > > > Description > ------- > > Added test cases for PARTITION_AWARE behavior. > > > Diffs > ----- > > src/tests/partition_tests.cpp f3142ad8d50daafcdb70ad9dbb2772f8ba30db00 > > Diff: https://reviews.apache.org/r/50706/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Neil Conway > >