> On Aug. 21, 2017, 4:33 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/paths.hpp > > Lines 54 (patched) > > > > > > Align this field? Ah whoops. Was: ``` // | |-- // | |-- pid ``` Should be: ``` // | | |-- // | |-- pid ``` - Joseph ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61806/#review183411 ----------------------------------------------------------- On Aug. 21, 2017, 3:21 p.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61806/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2017, 3:21 p.m.) > > > Review request for mesos, Gilbert Song and Jie Yu. > > > Bugs: MESOS-7305 > https://issues.apache.org/jira/browse/MESOS-7305 > > > Repository: mesos > > > Description > ------- > > This attempts to match the structure of the top level comments > in "slave/paths.hpp" and "slave/containerizer/mesos/paths.hpp". > > The runtime directory tree/drawing now shows all potential entries > that can exist in the file structure and the potential nesting of > containers. > > > Diffs > ----- > > src/slave/containerizer/mesos/paths.hpp 12ae7c7329f9e8d334cb32c30cc38465bddc046c > > > Diff: https://reviews.apache.org/r/61806/diff/1/ > > > Testing > ------- > > Comment change only. > > > Thanks, > > Joseph Wu > >