You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@mesos.apache.org by Benjamin Hindman <be...@berkeley.edu> on 2014/03/10 09:02:59 UTC

Review Request 18954: Added Future::after.

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

Review request for mesos, Ben Mahler, Jie Yu, and Vinod Kone.


Repository: mesos-git


Description
-------

This is to avoid a standard pattern seen with doing a 'delay' after some asynchronous function call has been made and we're waiting on it's completion via Future::then, onAny, etc. This allows one to do:

someAsynchronousFunction()
  .then(defer(self(), &Self::_continuation1))
  .then(defer(self(), &Self::_continuation2))
  .after(Seconds(30), defer(self(), &Self::_timeout, lambda::_1));


Diffs
-----

  3rdparty/libprocess/include/process/future.hpp 27b0970bf1d1ae1b977ddfc2de5ee858f1031bf5 
  3rdparty/libprocess/include/process/http.hpp 7f549ba3476ecf5dec0db21d57ee58bcd73d5996 
  3rdparty/libprocess/src/tests/process_tests.cpp e899aed7dbe6e8645484fe0ba69521e9fb0fdad2 

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


Testing
-------

make check


Thanks,

Benjamin Hindman