[
https://issues.apache.org/jira/browse/FLINK-10400?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16628923#comment-16628923
]
ASF GitHub Bot commented on FLINK-10400:
----------------------------------------
tillrohrmann commented on a change in pull request #6742: [FLINK-10400] Fail
JobResult if application finished in CANCELED or FAILED state
URL: https://github.com/apache/flink/pull/6742#discussion_r220605439
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/JobResult.java
##########
@@ -108,22 +118,39 @@ public long getNetRuntime() {
*
* @param classLoader to use for deserialization
* @return JobExecutionResult
- * @throws WrappedJobException if the JobResult contains a serialized
exception
+ * @throws JobExecutionException if the job execution did not succeed
Review comment:
Good point @TisonKun. Will add it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Return failed JobResult if job terminates in state FAILED or CANCELED
> ---------------------------------------------------------------------
>
> Key: FLINK-10400
> URL: https://issues.apache.org/jira/browse/FLINK-10400
> Project: Flink
> Issue Type: Bug
> Components: Client
> Affects Versions: 1.6.1, 1.7.0, 1.5.4
> Reporter: Till Rohrmann
> Assignee: Till Rohrmann
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.7.0, 1.6.2, 1.5.5
>
>
> If the job reaches the globally terminal state {{FAILED}} or {{CANCELED}},
> the {{JobResult}} must return a non-successful result. At the moment, it can
> happen that in the {{CANCELED}} state where we don't find a failure cause
> that we return a successful {{JobResult}}.
> In order to change this I propose to always return a {{JobResult}} with a
> {{JobCancellationException}} in case of {{CANCELED}} and a
> {{JobExecutionException}} in case of {{FAILED}}.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
