Re: Review Request 44232: Added a check when umounting persistent volumes in docker containerizer.

2016-03-01 Thread Jie Yu
> On March 2, 2016, 1:31 a.m., Joseph Wu wrote: > > src/slave/containerizer/docker.cpp, lines 585-586 > > > > > > It would be a slightly stronger check if you include this as well: > > ``` > >

Re: Review Request 44232: Added a check when umounting persistent volumes in docker containerizer.

2016-03-01 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44232/#review121554 --- Ship it! Ship It! - Timothy Chen On March 1, 2016, 11:50

Re: Review Request 44232: Added a check when umounting persistent volumes in docker containerizer.

2016-03-01 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44232/ --- (Updated March 1, 2016, 11:50 p.m.) Review request for mesos, Joseph Wu and

Review Request 44232: Added a check when umounting persistent volumes in docker containerizer.

2016-03-01 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44232/ --- Review request for mesos, Joseph Wu and Timothy Chen. Bugs: MESOS-4832