Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-06-13 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/#review45665 --- Ship it! Ship It! - Vinod Kone On June 12, 2014, 9:50 p.m., Ian

Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-06-12 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/ --- (Updated June 12, 2014, 9:50 p.m.) Review request for mesos and Vinod Kone. C

Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-06-09 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/#review45064 --- Patch looks great! Reviews applied: [21442] All tests passed. - M

Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-06-08 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/ --- (Updated June 8, 2014, 10:08 p.m.) Review request for mesos and Benjamin Hindma

Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-05-30 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/#review8 --- Patch looks great! Reviews applied: [21442] All tests passed. - M

Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-05-29 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/ --- (Updated May 29, 2014, 3:18 p.m.) Review request for mesos and Benjamin Hindman

Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-05-29 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/ --- (Updated May 29, 2014, 2:25 p.m.) Review request for mesos and Benjamin Hindman

Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-05-29 Thread Ian Downes
> On May 15, 2014, 11:33 a.m., Vinod Kone wrote: > > include/mesos/mesos.proto, line 446 > > > > > > s/Epoch/epoch/ ? Capitalization is correct, see man date: Print the date and time represented by seconds, where sec

Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-05-28 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/#review44234 --- Patch looks great! Reviews applied: [21442] All tests passed. - M

Re: Review Request 21442: Introduce a PerfStatistics protobuf.

2014-05-16 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/#review43137 --- ContainerStatistics is a great idea. How big of a change is it? in

Review Request 21442: Introduce a PerfStatistics protobuf.

2014-05-14 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21442/ --- Review request for mesos and Benjamin Hindman. Bugs: MESOS-1278 https://iss