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

Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone.


Repository: mesos-git


Description
-------

We currently 'defer' in order to acknowledge processed status updates in the 
scheduler driver.

This is unfortunate because it adds unnecessary delay and unnecessary retries, 
especially when the scheduler's queue is backed up.

It is sufficient to use 'aborted' to see if we can send a status update. 
Ideally there would be proper locking on this variable rather than the use of 
'volatile', but since we're mostly concerned about the driver aborting on the 
same thread as the status update was processed, I've left it as is.


Diffs
-----

  src/sched/sched.cpp 3684cfe9d153cab5f4ea86094fffd814ce74baa1 

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


Testing
-------

make check


Thanks,

Ben Mahler

Reply via email to