Re: Review Request 15853: Fixed scheduler driver to minimize sending duplicate status updates to the scheduler.

2014-02-17 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15853/#review34671 --- Ship it! Looks good to me. Would this also be a problem for the

Re: Review Request 15853: Fixed scheduler driver to minimize sending duplicate status updates to the scheduler.

2014-02-17 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15853/#review34678 --- Ship it! Ship It! - Niklas Nielsen On Feb. 14, 2014, 2:28 p.m.,

Re: Review Request 15853: Fixed scheduler driver to minimize sending duplicate status updates to the scheduler.

2014-02-14 Thread Vinod Kone
On Nov. 27, 2013, 4:12 a.m., Ben Mahler wrote: src/sched/sched.cpp, line 711 https://reviews.apache.org/r/15853/diff/1/?file=391118#file391118line711 Do we really need to log the framework id? It's nice to have when running multiple schedulers in tests. - Vinod

Re: Review Request 15853: Fixed scheduler driver to minimize sending duplicate status updates to the scheduler.

2014-02-14 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15853/ --- (Updated Feb. 14, 2014, 10:28 p.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 15853: Fixed scheduler driver to minimize sending duplicate status updates to the scheduler.

2013-11-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15853/#review29473 --- I've sent out https://reviews.apache.org/r/15872 to address the

Review Request 15853: Fixed scheduler driver to minimize sending duplicate status updates to the scheduler.

2013-11-25 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15853/ --- Review request for mesos, Benjamin Hindman and Ben Mahler. Bugs: MESOS-551

Re: Review Request 15853: Fixed scheduler driver to minimize sending duplicate status updates to the scheduler.

2013-11-25 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15853/#review29418 --- Handing this patch over to BenMahler because there is an issue with