Re: Review Request 27923: Use Slave::getExecutorInfo() only once.

2014-11-24 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27923/ --- (Updated Nov. 24, 2014, 11:33 a.m.) Review request for mesos and Ben Mahler.

Re: Review Request 27923: Use Slave::getExecutorInfo() only once.

2014-11-18 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27923/ --- (Updated Nov. 18, 2014, noon) Review request for mesos and Ben Mahler.

Re: Review Request 27923: Use Slave::getExecutorInfo() only once.

2014-11-18 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27923/#review62051 --- I don't think you looked over your diff before you published.. ;)

Re: Review Request 27923: Use Slave::getExecutorInfo() only once.

2014-11-14 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27923/#review61480 --- src/tests/mesos.cpp

Re: Review Request 27923: Use Slave::getExecutorInfo() only once.

2014-11-14 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27923/#review61481 --- Ship it! Ship It! - Ben Mahler On Nov. 12, 2014, 8:57 p.m., Ian

Re: Review Request 27923: Use Slave::getExecutorInfo() only once.

2014-11-13 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27923/#review61225 --- Ship it! Ship It! - Alexander Rukletsov On Nov. 12, 2014, 8:57

Review Request 27923: Use Slave::getExecutorInfo() only once.

2014-11-12 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27923/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-1718