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

(Updated Jan. 17, 2018, 9:37 a.m.)


Review request for mesos and Vinod Kone.


Changes
-------

Minor change.


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


Repository: mesos


Description
-------

Previously we only detach the following paths when the gc for the
executor's sandbox succeeds.
  1. /agent_workdir/frameworks/FID/executors/EID/runs/CID
  2. /agent_workdir/frameworks/FID/executors/EID/runs/latest
  3. /frameworks/FID/executors/EID/runs/latest

But the problem is, such gc may not always succeed, e.g., it may fail
due to the parent directory of the executor's sandbox already gc'ed.

Now in this patch, we will detach those paths regardless of the result
of gc.


Diffs (updated)
-----

  src/slave/slave.cpp 5ad6410c06ed7bfdf2e358c2737cfbad3e48df5d 


Diff: https://reviews.apache.org/r/65156/diff/3/

Changes: https://reviews.apache.org/r/65156/diff/2-3/


Testing
-------


Thanks,

Qian Zhang

Reply via email to