[ https://issues.apache.org/jira/browse/FLINK-19927?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17225375#comment-17225375 ]
Andrey Zagrebin commented on FLINK-19927: ----------------------------------------- After offline discussion with [~chesnay] and [~trohrmann], the decision is to proceed with the quick fix of the original intention in FLINK-17075 to stop tracking in EG#notifyExecutionChange, also for SchedulerNG. The refactoring suggestions to move tracking to DefaultScheduler are moved to the follow-up ticket FLINK-19954. > ExecutionStateUpdateListener is only updated when legacy scheduling is enabled > ------------------------------------------------------------------------------ > > Key: FLINK-19927 > URL: https://issues.apache.org/jira/browse/FLINK-19927 > Project: Flink > Issue Type: Bug > Components: Runtime / Coordination > Affects Versions: 1.12.0 > Reporter: Robert Metzger > Assignee: Andrey Zagrebin > Priority: Blocker > Fix For: 1.12.0 > > > This is a finding from FLINK-19805. > The {{ExecutionDeploymentTracker}} is never notified about executions > reaching terminal state, when using the default scheduler. > This can potentially lead to invalid execution reconciliation behavior. > Fixing this ticket probably involves switching the statements here: > https://github.com/apache/flink/blob/master/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ExecutionGraph.java#L1688-L1692 > As part of the this tickets resolution, I suggest to also introduce a test > case. -- This message was sent by Atlassian Jira (v8.3.4#803005)