----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7756/#review12872 -----------------------------------------------------------
src/linux/cgroups.cpp <https://reviews.apache.org/r/7756/#comment27675> hmm..why the revert? src/linux/cgroups.cpp <https://reviews.apache.org/r/7756/#comment27678> I understand this code is a copy paste from initialize(), but I don't follow the flow here. Can someone explain? src/linux/cgroups.cpp <https://reviews.apache.org/r/7756/#comment27677> s/in/is in/ - Vinod Kone On Oct. 29, 2012, 5:45 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7756/ > ----------------------------------------------------------- > > (Updated Oct. 29, 2012, 5:45 a.m.) > > > Review request for mesos, Benjamin Hindman and Jie Yu. > > > Description > ------- > > Parent Review: https://reviews.apache.org/r/7712/ > > We'd like to retry the freeze, kill, thaw, watchEmpty steps when the freezing > of the cgroups fails. > > This simply retries the above steps when the cgroup does not become empty in > the given time interval. > > > Diffs > ----- > > src/linux/cgroups.hpp 8147919 > src/linux/cgroups.cpp a6056c3 > > Diff: https://reviews.apache.org/r/7756/diff/ > > > Testing > ------- > > make check on ubuntu > > This version punts on testing for now, but testing suggestions welcome! > > > Thanks, > > Ben Mahler > >
