-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15597/
-----------------------------------------------------------

(Updated Nov. 18, 2013, 7:24 p.m.)


Review request for mesos, Benjamin Hindman and Vinod Kone.


Changes
-------

Removed the sleep in CommandExecutor per Vinod's review.


Bugs: MESOS-243
    https://issues.apache.org/jira/browse/MESOS-243


Repository: mesos-git


Description
-------

See MESOS-243, currently driver.join() will not wait for outstanding requests. 
This ensures any outstanding requests when driver.{stop|abort}() is called will 
be flushed prior to driver.join() returning. See sched.cpp for how this mirror 
the SchedulerDriver functionality.


Diffs (updated)
-----

  src/exec/exec.cpp c866838c90901610803df026329060c170d9f21a 
  src/launcher/executor.cpp b73ab479500a7347a38ba53acecfab9229f1080d 

Diff: https://reviews.apache.org/r/15597/diff/


Testing
-------

make check (surprised none of our tests sleep to compensate..?)


Thanks,

Ben Mahler

Reply via email to