Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-26 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47069/#review134940 --- Ship it! Ship It! - Michael Park On May 26, 2016, 4:43

Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-25 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47069/ --- (Updated May 26, 2016, 4:43 a.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-24 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47069/ --- (Updated May 24, 2016, 4:06 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-21 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47069/#review134250 --- src/common/http.cpp (lines 288 - 290)

Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-21 Thread Joerg Schad
> On May 21, 2016, 5:49 a.m., Michael Park wrote: > > Do we not need to update other parts such as `bool operator==(const Task&, > > const Task&);`, and exposing the `user` field in `/state`, etc? Nice catch, added. - Joerg --- This

Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-21 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47069/ --- (Updated May 21, 2016, 8:32 a.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-20 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47069/#review134233 --- Do we not need to update other parts such as `bool

Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-17 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47069/ --- (Updated May 17, 2016, 9:28 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-17 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47069/ --- (Updated May 17, 2016, 8:49 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 47069: Added `user` field to `Task` protobuf message.

2016-05-09 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47069/#review132246 --- In the description: `/protobug/protobuf/`, also, I would avoid