Changed return type of bulk executors and added test #1538
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The default return type of
bulk_execute
andbulk_async_execute
was changed fromvoid
andhpx::future<void>
tostd::vector<return_of_F_type>
(void if return_of_F_type is void) andstd::vector<hpx::future<return_of_F_type> >
for each of the executors and for the behavior of executor_traits. Note that the return type is forwarded from the executor_type so a user can easily create an executor with the behavior described by n4406. This was tested in created_executor.cpp.For sequential_executor, I don't think hpx::util::unwrapped is optimal in bulk_execute. Any suggestions? Also, executor tests compiled on my machine but running thread_pool_executors_test failed about half of the time. I got
{what}: assertion 'running()' failed: HPX(assertion_failure)