----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51275/#review146432 -----------------------------------------------------------
Fix it, then Ship it! src/linux/cgroups.cpp (line 1090) <https://reviews.apache.org/r/51275/#comment212861> Kill this line to be consistent with `exists` check above. src/linux/cgroups.cpp (line 1097) <https://reviews.apache.org/r/51275/#comment212862> Ditto. - Jie Yu On Aug. 22, 2016, 4:54 a.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51275/ > ----------------------------------------------------------- > > (Updated Aug. 22, 2016, 4:54 a.m.) > > > Review request for mesos and Jie Yu. > > > Repository: mesos > > > Description > ------- > > Cleans up code in LinuxLauncher. Adds a useful function for others to > use other places if necessary. > > > Diffs > ----- > > src/linux/cgroups.hpp 047d3ea596660e704447c0f0c7b914a43c4a2187 > src/linux/cgroups.cpp f3232c009d04bb82701c0407b12abf999ab60a73 > src/slave/containerizer/mesos/linux_launcher.cpp > 7377316776646e3d660086da3c0d5b494ce8ace4 > > Diff: https://reviews.apache.org/r/51275/diff/ > > > Testing > ------- > > > Thanks, > > Benjamin Hindman > >
