Re: Review Request 44571: Added timeout for destroying Docker containers.

2016-04-08 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44571/#review127841 --- Ship it! Just renamed the constant and clarified the comment a

Re: Review Request 44571: Added timeout for destroying Docker containers.

2016-04-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44571/#review127808 --- Patch looks great! Reviews applied: [44571] Passed command: expo

Re: Review Request 44571: Added timeout for destroying Docker containers.

2016-04-08 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44571/ --- (Updated April 8, 2016, 1:20 p.m.) Review request for mesos, Jie Yu and Joris V

Re: Review Request 44571: Added timeout for destroying Docker containers.

2016-04-07 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44571/#review127685 --- Fix it, then Ship it! src/slave/containerizer/docker.cpp (line

Re: Review Request 44571: Added timeout for destroying Docker containers.

2016-04-04 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44571/#review126848 --- Patch looks great! Reviews applied: [44571] Passed command: expo

Re: Review Request 44571: Added timeout for destroying Docker containers.

2016-04-04 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44571/ --- (Updated April 4, 2016, 4:05 p.m.) Review request for mesos, Jie Yu and Joris V

Re: Review Request 44571: Added timeout for destroying Docker containers.

2016-03-31 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44571/#review126330 --- Bad patch! Reviews applied: [44571] Failed command: ./support/ap

Re: Review Request 44571: Added timeout for destroying Docker containers.

2016-03-31 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44571/ --- (Updated March 31, 2016, 3:26 p.m.) Review request for mesos and Joris Van Remo