----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51651/#review147832 -----------------------------------------------------------
Patch looks great! Reviews applied: [51651] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On Sept. 6, 2016, 12:42 p.m., Kevin Klues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51651/ > ----------------------------------------------------------- > > (Updated Sept. 6, 2016, 12:42 p.m.) > > > Review request for mesos, Gilbert Song, Jie Yu, and Joseph Wu. > > > Bugs: MESOS-6100 and MESOS-6118 > https://issues.apache.org/jira/browse/MESOS-6100 > https://issues.apache.org/jira/browse/MESOS-6118 > > > Repository: mesos > > > Description > ------- > > Previously, we were using a recursive lambda with captured state > as part of the algorithm for sorting the entries in the mount info > table. In certain edge cases, it appeared that doing things this way > may have caused memory corruption that was difficult to reason about. > > To avoid this, we have changed the recursive algorithm to an iterative > one. As a side effect, the actual ordering of the entries is now > different with the new algorithm (the original algorithm did a > depth-first traversal of the mount hierarchy and the new one does a > breadth-first traversal). However, both algorithms maintain the same > invariant -- all parent mount points appear earlier in the sorted list > than their children. > > > Diffs > ----- > > src/linux/fs.cpp 14ae5a9089916549f691363bc2269e13c5260a14 > > Diff: https://reviews.apache.org/r/51651/diff/ > > > Testing > ------- > > `GTEST_FILTER="*MountInfoTableReadSorted*" make -j24 check` > > I'm currently running it through the CI. > > I will update this section after eveything passes. > > > Thanks, > > Kevin Klues > >
