> On Feb. 27, 2013, 6:43 p.m., Ben Mahler wrote: > > src/slave/cgroups_isolation_module.cpp, line 946 > > <https://reviews.apache.org/r/9408/diff/12/?file=261997#file261997line946> > > > > As a result of this NOTE it sounds like we should kill the sentinel > > values in favor of using Options, not expecting you to do that but maybe a > > TODO in the info struct if not already one present. > > Vinod Kone wrote: > As the TODO notes the plan is to actually wait for the process terminated > event, before informing the slave. So, need to make this sentinel.
s/need to/no need to/ - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9408/#review17156 ----------------------------------------------------------- On Feb. 26, 2013, 12:32 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9408/ > ----------------------------------------------------------- > > (Updated Feb. 26, 2013, 12:32 a.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > Description > ------- > > See summary > > > Diffs > ----- > > src/slave/cgroups_isolation_module.hpp > a04fc46b15d2741886f5847cadbdc9bed351c588 > src/slave/cgroups_isolation_module.cpp > a779de80d13c67e507d7d2ee788fcdaa5e71daca > src/slave/slave.hpp 7648c33230c1900eda7529045c5df9ccab105d47 > src/slave/slave.cpp 8c2e1bfc363491c681177676f9dfe5f229276f7d > src/tests/balloon_framework_test.sh > 93a733f64cfde08349b7781eb3d5e13594c74498 > > Diff: https://reviews.apache.org/r/9408/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
