> On Aug. 23, 2016, 7:54 a.m., Qian Zhang wrote: > > src/slave/containerizer/mesos/isolators/cgroups/subsystem.hpp, line 99 > > <https://reviews.apache.org/r/51293/diff/1/?file=1480783#file1480783line99> > > > > I think we are not returning `Nothing`, instead we are returning > > `ContainerLimitation` on success, right?
Thanks a lot! Fixed. - haosdent ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51293/#review146486 ----------------------------------------------------------- On Aug. 22, 2016, 4:11 p.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51293/ > ----------------------------------------------------------- > > (Updated Aug. 22, 2016, 4:11 p.m.) > > > Review request for mesos, Gilbert Song, Jie Yu, and Qian Zhang. > > > Repository: mesos > > > Description > ------- > > Fixed a typo in the comment of `Subsystem::watch`. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/cgroups/subsystem.hpp > 6dca38cfdd360288c03a7a01aebbf15e3cbc5dce > > Diff: https://reviews.apache.org/r/51293/diff/ > > > Testing > ------- > > > Thanks, > > haosdent huang > >
