[jira] [Created] (FLINK-22060) Move null handling from ErrorInfo into Task.notifyFinalState

["Matthias (Jira)" <ji...@apache.org>]

Matthias created FLINK-22060:
--------------------------------

             Summary: Move null handling from ErrorInfo into Task.notifyFinalState
                 Key: FLINK-22060
                 URL: https://issues.apache.org/jira/browse/FLINK-22060
             Project: Flink
          Issue Type: Bug
            Reporter: Matthias


Ideally, we would want to have this {{null}} check since a failure should always have a cause. We didn't introduce a null check so far because of FLINK-21376. There is [ErrorInfo. createErrorInfoWithNullableCause|https://github.com/apache/flink/blob/c77a686c195d1742c276f4a9e75899c8b85377bb/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ErrorInfo.java#L50] for handling this. But it feels to be handled in the wrong place. Instead, we should substitute the {{null}} value by the {{FlinkException}} in [Task.notifyFinalState|https://github.com/apache/flink/blob/master/flink-runtime/src/main/java/org/apache/flink/runtime/taskmanager/Task.java#L1003]. This way, we make sure that {{null}} never ends up in the {{ExecutionGraph}}-related code which enables us to make the invariants stricter on our end.

This would serve as a preparation task



--
This message was sent by Atlassian Jira
(v8.3.4#803005)