Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-04-08 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/#review79404 --- Ship it! Ship It! - Jie Yu On April 7, 2015, 9:52 p.m., Michael

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-04-07 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/ --- (Updated April 7, 2015, 9:51 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-04-07 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/ --- (Updated April 7, 2015, 9:52 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-03-25 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/#review1 --- include/mesos/mesos.proto

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-03-25 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/ --- (Updated March 25, 2015, 6:47 p.m.) Review request for mesos, Alexander Ruklets

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-03-25 Thread Michael Park
> On March 24, 2015, 7:25 a.m., Timothy Chen wrote: > > include/mesos/mesos.proto, line 397 > > > > > > Not sure this is the right place to comment on the valid specifications > > of both role and reservation. There i

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-03-25 Thread Michael Park
> On March 24, 2015, 7:25 a.m., Timothy Chen wrote: > > include/mesos/mesos.proto, line 397 > > > > > > Not sure this is the right place to comment on the valid specifications > > of both role and reservation. There i

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-03-24 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/#review77536 --- include/mesos/mesos.proto

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-03-18 Thread Michael Park
> On March 18, 2015, 12:27 a.m., Alexander Rukletsov wrote: > > include/mesos/mesos.proto, lines 412-415 > > > > > > Are you saying "Resources cannot be reserved for '*' role"? Or there is > > more information that it

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-03-18 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/ --- (Updated March 18, 2015, 7:43 p.m.) Review request for mesos, Alexander Ruklets

Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-03-17 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/#review76764 --- include/mesos/mesos.proto

Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.

2015-03-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/ --- Review request for mesos, Alexander Rukletsov, Ben Mahler, and Jie Yu. Bugs: ME