----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47336/#review133158 -----------------------------------------------------------
Ship it! Looks fine. Consider switching to ``unique_ptr``, since we only check the ``fts_close`` error 1 of 3 times. - James Peach On May 13, 2016, 7:09 a.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47336/ > ----------------------------------------------------------- > > (Updated May 13, 2016, 7:09 a.m.) > > > Review request for mesos and Timothy Chen. > > > Repository: mesos > > > Description > ------- > > Fixed FTS handle leak issue in xfs isolator. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/xfs/utils.cpp > 80404c396ae98622a69ec290881226c1753d3626 > > Diff: https://reviews.apache.org/r/47336/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > Guangya Liu > >
