Re: Review Request 62644: Added resources to the TaskStatus message.

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

Re: Review Request 62644: Added resources to the TaskStatus message.

2017-10-11 Thread James Peach
> On Oct. 11, 2017, 12:48 p.m., Qian Zhang wrote: > > src/slave/slave.cpp > > Lines 7043-7044 (patched) > > > > > > These two lines can be merged into a single line. Yes but if you do that, the indentation of the

Re: Review Request 62644: Added resources to the TaskStatus message.

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

Re: Review Request 62644: Added resources to the TaskStatus message.

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

Re: Review Request 62644: Added resources to the TaskStatus message.

2017-10-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62644/#review186968 --- src/slave/slave.cpp Lines 6962-6965 (patched)

Re: Review Request 62644: Added resources to the TaskStatus message.

2017-10-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62644/#review186964 --- Ship it! Ship It! - Jie Yu On Sept. 28, 2017, 12:19 a.m.,

Review Request 62644: Added resources to the TaskStatus message.

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