You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Kevin Klues <kl...@gmail.com> on 2016/01/29 11:08:49 UTC
Review Request 42955: Added variadic template for process::collect.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42955/
-----------------------------------------------------------
Review request for mesos, Joris Van Remoortere and Michael Park.
Repository: mesos
Description
-------
Previously, templates only existed to allow collect to take either a
std::list of Futures, or *exactly* 2 Futures as arguments.
This commit removes the 2 argument template and replaces it with a
variadic template that accepts an arbitrary number of Futures as
arguments.
Diffs
-----
3rdparty/libprocess/include/process/collect.hpp cd78b6c211c2e4ca2b1ebbe728cfc2dfad1a32c9
Diff: https://reviews.apache.org/r/42955/diff/
Testing
-------
Thanks,
Kevin Klues
Re: Review Request 42955: Added variadic template for
process::collect.
Posted by Kevin Klues <kl...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42955/
-----------------------------------------------------------
(Updated Jan. 29, 2016, 10:10 a.m.)
Review request for mesos, Joris Van Remoortere and Michael Park.
Bugs: MESOS-4551
https://issues.apache.org/jira/browse/MESOS-4551
Repository: mesos
Description
-------
Previously, templates only existed to allow collect to take either a
std::list of Futures, or *exactly* 2 Futures as arguments.
This commit removes the 2 argument template and replaces it with a
variadic template that accepts an arbitrary number of Futures as
arguments.
Diffs
-----
3rdparty/libprocess/include/process/collect.hpp cd78b6c211c2e4ca2b1ebbe728cfc2dfad1a32c9
Diff: https://reviews.apache.org/r/42955/diff/
Testing
-------
Thanks,
Kevin Klues