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 2012/05/03 01:29:12 UTC

Review Request: Added Future::then.

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

Review request for mesos and John Sirois.


Summary
-------

See summary.


Diffs
-----

  third_party/libprocess/include/process/future.hpp ecee501 
  third_party/libprocess/src/tests.cpp e6af7a4 

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


Testing
-------

make check
$ pwd
/path/to/mesos/build/third_party/libprocess
$ ./tests


Thanks,

Benjamin


Re: Review Request: Added Future::then.

Posted by Benjamin Hindman <be...@berkeley.edu>.

> On 2012-05-03 18:21:21, John Sirois wrote:
> >

Thanks, submitted.


- Benjamin


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


On 2012-05-02 23:29:12, Benjamin Hindman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/4980/
> -----------------------------------------------------------
> 
> (Updated 2012-05-02 23:29:12)
> 
> 
> Review request for mesos and John Sirois.
> 
> 
> Summary
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   third_party/libprocess/include/process/future.hpp ecee501 
>   third_party/libprocess/src/tests.cpp e6af7a4 
> 
> Diff: https://reviews.apache.org/r/4980/diff
> 
> 
> Testing
> -------
> 
> make check
> $ pwd
> /path/to/mesos/build/third_party/libprocess
> $ ./tests
> 
> 
> Thanks,
> 
> Benjamin
> 
>


Re: Review Request: Added Future::then.

Posted by John Sirois <jo...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4980/#review7509
-----------------------------------------------------------

Ship it!


- John


On 2012-05-02 23:29:12, Benjamin Hindman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/4980/
> -----------------------------------------------------------
> 
> (Updated 2012-05-02 23:29:12)
> 
> 
> Review request for mesos and John Sirois.
> 
> 
> Summary
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   third_party/libprocess/include/process/future.hpp ecee501 
>   third_party/libprocess/src/tests.cpp e6af7a4 
> 
> Diff: https://reviews.apache.org/r/4980/diff
> 
> 
> Testing
> -------
> 
> make check
> $ pwd
> /path/to/mesos/build/third_party/libprocess
> $ ./tests
> 
> 
> Thanks,
> 
> Benjamin
> 
>