Re: Review Request 36071: Add flow diagram for docker containerizer.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36071/#review141548 --- Closing this review due to inactivity. Please see our

Re: Review Request 36071: Add flow diagram for docker containerizer.

2016-02-18 Thread Bernd Mathiske
> On July 7, 2015, 4:51 a.m., Bernd Mathiske wrote: > > 1. Inconsistent capitalization in box labels. > > 2. You explain different paths to obtain an "executor pid" and then you > > checkpoint a "container pid". You lost me there. > > 3. What happens to the tasks? Is this diagram for the

Re: Review Request 36071: Add flow diagram for docker containerizer.

2015-11-04 Thread Bernd Mathiske
> On July 7, 2015, 4:51 a.m., Bernd Mathiske wrote: > > 1. Inconsistent capitalization in box labels. > > 2. You explain different paths to obtain an "executor pid" and then you > > checkpoint a "container pid". You lost me there. > > 3. What happens to the tasks? Is this diagram for the

Re: Review Request 36071: Add flow diagram for docker containerizer.

2015-07-07 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36071/#review90685 --- 1. Inconsistent capitalization in box labels. 2. You explain

Re: Review Request 36071: Add flow diagram for docker containerizer.

2015-07-02 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36071/#review90242 --- This diagram does seem to be missing some essential

Re: Review Request 36071: Add flow diagram for docker containerizer.

2015-07-01 Thread Kapil Arya
On July 1, 2015, 11:45 a.m., Till Toenshoff wrote: When trying to apply locally, I get: ``` $ ./support/apply-review.sh 36071 2015-07-01 17:43:28 URL:https://reviews.apache.org/r/36071/diff/raw/ [1338/1338] - 36071.patch [1] 36071.patch:15: new blank line at EOF. + error:

Re: Review Request 36071: Add flow diagram for docker containerizer.

2015-07-01 Thread Timothy Chen
On July 1, 2015, 3:45 p.m., Till Toenshoff wrote: When trying to apply locally, I get: ``` $ ./support/apply-review.sh 36071 2015-07-01 17:43:28 URL:https://reviews.apache.org/r/36071/diff/raw/ [1338/1338] - 36071.patch [1] 36071.patch:15: new blank line at EOF. + error:

Re: Review Request 36071: Add flow diagram for docker containerizer.

2015-07-01 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36071/#review90052 --- When trying to apply locally, I get: ``` $

Review Request 36071: Add flow diagram for docker containerizer.

2015-06-30 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36071/ --- Review request for mesos, Benjamin Hindman, Bernd Mathiske, and Till Toenshoff.

Re: Review Request 36071: Add flow diagram for docker containerizer.

2015-06-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36071/#review90010 --- Bad patch! Reviews applied: [36071] Failed command: