> On Jan. 28, 2015, 8:10 p.m., Jie Yu wrote: > > src/master/master.cpp, line 2904 > > <https://reviews.apache.org/r/30361/diff/1/?file=838578#file838578line2904> > > > > I guess the same issue in my patch. We cannot assume resources haven't > > been persisted. Let me think about a generic way to solve this problem.
As I mentioned on IRC, for `RESERVE` and `UNRESERVE` we can actually assume such things. Left comments for both cases. - Michael ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30361/#review70066 ----------------------------------------------------------- On Feb. 5, 2015, 12:58 a.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30361/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2015, 12:58 a.m.) > > > Review request for mesos, Adam B, Benjamin Hindman, Ben Mahler, Jie Yu, and > Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Implemented logic to handle dynamic reservation operations in Master::_accept. > > > Diffs > ----- > > src/master/master.cpp 234bbecc4205036d790b026abd59100eb188f913 > src/master/validation.hpp 2d7416c053f82d6316542fa9c35b0e7bc605abec > src/master/validation.cpp 2c4b16df1ffe34c7e2a60a3d428242da4c20c347 > > Diff: https://reviews.apache.org/r/30361/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Michael Park > >
