Re: Review Request 37921: Add Copy backend for provisioners.

2015-09-02 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/ --- (Updated Sept. 2, 2015, 4:54 p.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 37921: Add Copy backend for provisioners.

2015-09-02 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/ --- (Updated Sept. 3, 2015, 1:24 a.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 37921: Add Copy backend for provisioners.

2015-09-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/#review97508 --- Patch looks great! Reviews applied: [37921] All tests passed. -

Re: Review Request 37921: Add Copy backend for provisioners.

2015-09-01 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/#review97376 --- src/slave/containerizer/provisioners/backends/copy.cpp (lines 105

Re: Review Request 37921: Add Copy backend for provisioners.

2015-09-01 Thread Timothy Chen
> On Sept. 2, 2015, 3:07 a.m., Jiang Yan Xu wrote: > > src/slave/containerizer/provisioners/backends/copy.cpp, lines 108-116 > > > > > > Wow, I guess I hadn't realized what you meant by "making sure the > >

Re: Review Request 37921: Add Copy backend for provisioners.

2015-08-31 Thread Timothy Chen
> On Aug. 31, 2015, 5:50 p.m., Jiang Yan Xu wrote: > > src/slave/containerizer/provisioners/backends/copy.cpp, line 19 > > > > > > Move this below. > > > > ``` > > #include "common/status_utils.hpp" > >

Re: Review Request 37921: Add Copy backend for provisioners.

2015-08-31 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/ --- (Updated Sept. 1, 2015, 12:56 a.m.) Review request for mesos, Jie Yu and Jiang

Re: Review Request 37921: Add Copy backend for provisioners.

2015-08-31 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/ --- (Updated Sept. 1, 2015, 12:58 a.m.) Review request for mesos, Jie Yu and Jiang

Review Request 37921: Add Copy backend for provisioners.

2015-08-29 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/ --- Review request for mesos, Jie Yu and Jiang Yan Xu. Repository: mesos

Re: Review Request 37921: Add Copy backend for provisioners.

2015-08-29 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/#review96982 --- Patch looks great! Reviews applied: [37921] All tests passed. -