Re: Review Request 56812: Updated agent handlers to use the 'Principal' type.

2017-03-03 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56812/#review167902 --- Ship it! Ship It! - Vinod Kone On March 3, 2017, 6:43 p.m.,

Re: Review Request 56812: Updated agent handlers to use the 'Principal' type.

2017-03-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56812/ --- (Updated March 3, 2017, 6:43 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 56812: Updated agent handlers to use the 'Principal' type.

2017-02-28 Thread Greg Mann
> On Feb. 24, 2017, 2:07 a.m., Vinod Kone wrote: > > src/slave/http.cpp, line 836 > > > > > > not related to your change, but I'm curious why this method uses the > > object approver whereas the

Re: Review Request 56812: Updated agent handlers to use the 'Principal' type.

2017-02-28 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56812/#review167058 --- Ship it! Ship It! - Alexander Rojas On Feb. 28, 2017, 7:37

Re: Review Request 56812: Updated agent handlers to use the 'Principal' type.

2017-02-28 Thread Alexander Rojas
> On Feb. 24, 2017, 3:07 a.m., Vinod Kone wrote: > > src/slave/http.cpp, line 836 > > > > > > not related to your change, but I'm curious why this method uses the > > object approver whereas the

Re: Review Request 56812: Updated agent handlers to use the 'Principal' type.

2017-02-27 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56812/ --- (Updated Feb. 28, 2017, 6:37 a.m.) Review request for mesos, Adam B, Alexander