Review Request 58251: Changed 'Principal.claims' to a hashmap.

2017-04-06 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58251/ --- Review request for mesos, Adam B, Alexander Rojas, Till Toenshoff, and Vinod Kon

Re: Review Request 58251: Changed 'Principal.claims' to a hashmap.

2017-04-07 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58251/#review171316 --- 3rdparty/libprocess/include/process/authenticator.hpp Line 69 (or

Re: Review Request 58251: Changed 'Principal.claims' to a hashmap.

2017-04-07 Thread Greg Mann
> On April 7, 2017, 7:47 a.m., Alexander Rojas wrote: > > 3rdparty/libprocess/include/process/authenticator.hpp > > Line 69 (original), 69 (patched) > > > > > > My only concern with using either a `std::map` or a `has

Re: Review Request 58251: Changed 'Principal.claims' to a hashmap.

2017-04-10 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58251/#review171477 --- Ship it! Ship It! - Vinod Kone On April 7, 2017, 3:26 a.m.,

Re: Review Request 58251: Changed 'Principal.claims' to a hashmap.

2017-04-12 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58251/#review171706 --- Ship it! Ship It! - Alexander Rojas On April 7, 2017, 5:26 a

Re: Review Request 58251: Changed 'Principal.claims' to a hashmap.

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