Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-19 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/ --- (Updated April 19, 2017, 10:48 p.m.) Review request for mesos, Adam B, Mesos Re

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-19 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/ --- (Updated April 19, 2017, 10:07 p.m.) Review request for mesos, Adam B, Mesos Re

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-19 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/ --- (Updated April 19, 2017, 10:05 p.m.) Review request for mesos, Adam B, Mesos Re

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-18 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/ --- (Updated April 18, 2017, 10:15 p.m.) Review request for mesos, Adam B, Mesos Re

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-17 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/#review172126 --- Ship it! Ship It! - Neil Conway On April 17, 2017, 4:14 p.m.

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-17 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/ --- (Updated April 17, 2017, 4:14 p.m.) Review request for mesos, Adam B, Mesos Rev

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-11 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/ --- (Updated April 11, 2017, 10:33 p.m.) Review request for mesos, Adam B, Mesos Re

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-11 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/#review171614 --- For consistency, should we also remove the `CHECK(slaves.register

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-11 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/ --- (Updated April 11, 2017, 4:27 p.m.) Review request for mesos and Mesos Reviewbo

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-11 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/#review171532 --- Fix it, then Ship it! LGTM, minor tweak to a log message src/

Re: Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-10 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/ --- (Updated April 10, 2017, 3:50 p.m.) Review request for mesos and Mesos Reviewbo

Review Request 58304: Remove unnecessary hashmap lookups.

2017-04-10 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58304/ --- Review request for mesos. Bugs: MESOS-7372 https://issues.apache.org/jira/b