Re: Review Request 62643: Propagated the container termination to the agent API.

2017-10-14 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62643/#review188079 --- Ship it! Ship It! - Qian Zhang On Oct. 10, 2017, 8:08 a.m.,

Re: Review Request 62643: Propagated the container termination to the agent API.

2017-10-11 Thread James Peach
> On Oct. 11, 2017, 12:18 p.m., Qian Zhang wrote: > > include/mesos/agent/agent.proto > > Lines 347 (patched) > > > > > > Can you elaborate a bit about why wrapping `repeated Resource resources > > = 4;` into a new

Re: Review Request 62643: Propagated the container termination to the agent API.

2017-10-11 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62643/#review187639 --- include/mesos/agent/agent.proto Lines 347 (patched)

Re: Review Request 62643: Propagated the container termination to the agent API.

2017-10-09 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62643/ --- (Updated Oct. 10, 2017, 12:08 a.m.) Review request for mesos, Jie Yu and Qian

Re: Review Request 62643: Propagated the container termination to the agent API.

2017-10-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62643/#review186962 --- Fix it, then Ship it! include/mesos/agent/agent.proto Lines

Review Request 62643: Propagated the container termination to the agent API.

2017-09-27 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62643/ --- Review request for mesos, Jie Yu and Qian Zhang. Bugs: MESOS-7963