> On April 21, 2016, 2:02 p.m., Vinod Kone wrote: > > src/examples/balloon_executor.cpp, line 149 > > <https://reviews.apache.org/r/46407/diff/1/?file=1351758#file1351758line149> > > > > I don't quite follow what the bug was previously that you are trying to > > fix here with a separate thread?
Added a comment alongside the task thread to explain. - Joseph ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46407/#review129955 ----------------------------------------------------------- On June 6, 2016, 4:48 p.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46407/ > ----------------------------------------------------------- > > (Updated June 6, 2016, 4:48 p.m.) > > > Review request for mesos, Greg Mann, Artem Harutyunyan, Kevin Klues, and > Vinod Kone. > > > Bugs: MESOS-5174 > https://issues.apache.org/jira/browse/MESOS-5174 > > > Repository: mesos > > > Description > ------- > > Fixes the logic in the example `balloon-executor` to work in cases not > exercised by the `ROOT_CGROUPS_BalloonFramework` test. > > * The "task" logic was moved into a separate thread. This fixes the > case where the balloon executor does not exceed the memory limit. > (i.e. by Unblocking the driver's thread while sending status updates.) > * Changed log messages to use glog. > * Added logic to prevent multiple task launches with the same executor. > > > Diffs > ----- > > src/examples/balloon_executor.cpp 2a79c353e96930d25495732d15b83c82247974bb > > Diff: https://reviews.apache.org/r/46407/diff/ > > > Testing > ------- > > See next review in this chain. > > > Thanks, > > Joseph Wu > >
