Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Future.wait doesn't give time to callers to install error-handlers. #27249

Closed
floitschG opened this issue Sep 5, 2016 · 1 comment

Comments

@floitschG
Copy link
Contributor

commented Sep 5, 2016

@lrhn

This comment has been minimized.

Copy link
Member

commented Sep 6, 2016

To be honest, I wouldn't mind if this case threw an error synchronously. It's an error in either iteration or calling Future.then and that just shouldn't happen. If it does happen, the error does not come from any of the futures, so putting it into the result future could be a little confusing.

On the other hand, having asynchronous functions report all their errors in the returned future seems to make some people happy.

@floitschG floitschG closed this in 70c48b9 Sep 6, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.