Re: Review Request 52009: Added slave path helper method 'getNestedSandboxPath()'.

2016-09-18 Thread Jie Yu 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52009/#review149392
---




src/slave/paths.hpp (line 206)


Why need this? Can we use a combination of getExecutorRunPath and 
getSandboxPath to achieve the same?


- Jie Yu


On Sept. 18, 2016, 5:31 p.m., Gilbert Song wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52009/
> ---
> 
> (Updated Sept. 18, 2016, 5:31 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Artem Harutyunyan, Jie Yu, Joseph 
> Wu, Kevin Klues, and Vinod Kone.
> 
> 
> Bugs: MESOS-6191
> https://issues.apache.org/jira/browse/MESOS-6191
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Added slave path helper method 'getNestedSandboxPath()'.
> 
> 
> Diffs
> -
> 
>   src/slave/paths.hpp ebff3a9fab3f4a8d38502fe02bd5a34cf95c1830 
>   src/slave/paths.cpp eb414caae5847949854743d4e3c603217a897898 
> 
> Diff: https://reviews.apache.org/r/52009/diff/
> 
> 
> Testing
> ---
> 
> make check
> 
> 
> Thanks,
> 
> Gilbert Song
> 
>

Review Request 52009: Added slave path helper method 'getNestedSandboxPath()'.

2016-09-18 Thread Gilbert Song 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52009/
---

Review request for mesos, Benjamin Hindman, Artem Harutyunyan, Jie Yu, Joseph 
Wu, Kevin Klues, and Vinod Kone.


Bugs: MESOS-6191
https://issues.apache.org/jira/browse/MESOS-6191


Repository: mesos


Description
---

Added slave path helper method 'getNestedSandboxPath()'.


Diffs
-

  src/slave/paths.hpp ebff3a9fab3f4a8d38502fe02bd5a34cf95c1830 
  src/slave/paths.cpp eb414caae5847949854743d4e3c603217a897898 

Diff: https://reviews.apache.org/r/52009/diff/


Testing
---

make check


Thanks,

Gilbert Song