[
https://issues.apache.org/jira/browse/FLINK-6050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15933015#comment-15933015
]
ASF GitHub Bot commented on FLINK-6050:
---------------------------------------
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/3537
Thanks for the review @StephanEwen. I will check whether we have such a
test case and if not, I'll add it.
> Improve failure reporting when using Future.thenAccept
> ------------------------------------------------------
>
> Key: FLINK-6050
> URL: https://issues.apache.org/jira/browse/FLINK-6050
> Project: Flink
> Issue Type: Improvement
> Components: Distributed Coordination
> Affects Versions: 1.3.0
> Reporter: Till Rohrmann
> Assignee: Till Rohrmann
> Priority: Minor
>
> When applying {{Future.thenAccept(Async)}} onto a {{Future}}, then we should
> register the exception handler on the returned {{Future<Void>}} and not on
> the original future. This has the advantage that we also catch exceptions
> which are thrown in the {{AcceptFunction}} and not only those originating
> from the original {{Future}}. This improve Flink's behaviour, because
> exceptions are not swallowed in the returned {{Future}}.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
