[ https://issues.apache.org/jira/browse/FLINK-5197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15705538#comment-15705538 ]
ASF GitHub Bot commented on FLINK-5197: --------------------------------------- GitHub user tillrohrmann opened a pull request: https://github.com/apache/flink/pull/2895 [FLINK-5197] [jm] Ignore outdated JobStatusChanged messages Outdated JobStatusChanged messages no longer trigger a RemoveJob message but are logged and ignored. This has the advantage, that an outdated JobStatusChanged message cannot interfere with a recovered job which can have the same job id. You can merge this pull request into a Git repository by running: $ git pull https://github.com/tillrohrmann/flink fixJobStatusChangedMessage Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/2895.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2895 ---- commit 490cef46380178a2296c2f743b9eb91154967463 Author: Till Rohrmann <trohrm...@apache.org> Date: 2016-11-29T15:02:29Z [FLINK-5197] [jm] Ignore outdated JobStatusChanged messages Outdated JobStatusChanged messages no longer trigger a RemoveJob message but are logged and ignored. This has the advantage, that an outdated JobStatusChanged message cannot interfere with a recovered job which can have the same job id. ---- > Late JobStatusChanged messages can interfere with running jobs > -------------------------------------------------------------- > > Key: FLINK-5197 > URL: https://issues.apache.org/jira/browse/FLINK-5197 > Project: Flink > Issue Type: Bug > Components: JobManager > Affects Versions: 1.2.0, 1.1.3 > Reporter: Till Rohrmann > Assignee: Till Rohrmann > Priority: Minor > Fix For: 1.2.0, 1.1.4 > > > When the {{JobManager}} receives a {{JobStatusChanged}} message, it will look > up the {{ExecutionGraph}} for the given {{JobID}}. If there is no > {{ExecutionGraph}}, then a {{RemoveJob}} message is sent to itself. In the > general case, this is not problematic, because the {{RemoveJob}} message > won't do anything if there is no {{ExecutionGraph}}. However, since this is > an asynchronous call, it can happen that the corresponding job of the > {{JobID}} is recovered before receiving the {{RemoveJob}} message. In this > case, the newly recovered job would be removed. > I propose to change the behaviour such that a {{JobStatusChanged}} for a > non-existing {{ExecutionGraph}} will be simply ignored. -- This message was sent by Atlassian JIRA (v6.3.4#6332)