Re: Review Request 45932: Add stripped TaskInfo's to ResourceUsage.Executor message.

> On April 9, 2016, 12:27 a.m., Ben Mahler wrote: > > include/mesos/mesos.proto, line 1032 > > > > > > Can we omit this? Why did you include it? I'll drop this as I don't have a good reason to use it for now. The r

Re: Review Request 45932: Add stripped TaskInfo's to ResourceUsage.Executor message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45932/ --- (Updated April 12, 2016, 5 a.m.) Review request for mesos and Ben Mahler. Cha

Re: Review Request 45932: Add stripped TaskInfo's to ResourceUsage.Executor message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45932/#review127933 --- Thanks! Looking pretty good, some minor comments below. CHANGELO

Re: Review Request 45932: Add stripped TaskInfo's to ResourceUsage.Executor message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45932/#review127898 --- Bad patch! Reviews applied: [45932, 45572] Failed command: ./sup