----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62047/#review185736 -----------------------------------------------------------
Ship it! src/slave/paths.hpp Lines 214 (patched) <https://reviews.apache.org/r/62047/#comment262057> // Returns the "virtual" path used to expose the executor's sandbox // via the /files endpoints: `/frameworks/FID/executors/EID/latest`. - Benjamin Mahler On Sept. 13, 2017, 1:25 a.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62047/ > ----------------------------------------------------------- > > (Updated Sept. 13, 2017, 1:25 a.m.) > > > Review request for mesos, Benjamin Mahler and Jason Lai. > > > Bugs: MESOS-7899 > https://issues.apache.org/jira/browse/MESOS-7899 > > > Repository: mesos > > > Description > ------- > > This patch added support to attach a virtual path > `/frameworks/{framework-id}/executors/{executor-id}/latest` to `Files` class > in agent, which can be used to navigate an executor's latest sandbox without > needing to know `work_dir` and `slave_id`. > > > Diffs > ----- > > src/slave/paths.hpp 51b481fc0870f1e95448f85ee2fd485fceea1919 > src/slave/paths.cpp 08177bcef63b998f691be6893ab35891098a2886 > src/slave/slave.cpp 6d1516a5d5b5db684f79385e60d892ff75fd00fd > > > Diff: https://reviews.apache.org/r/62047/diff/4/ > > > Testing > ------- > > Created new unit test in https://reviews.apache.org/r/62263/ > > > Thanks, > > Zhitao Li > >
