[
https://issues.apache.org/jira/browse/FLINK-3981?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15304733#comment-15304733
]
ASF GitHub Bot commented on FLINK-3981:
---------------------------------------
GitHub user mxm opened a pull request:
https://github.com/apache/flink/pull/2045
[FLINK-3981] don't log duplicate TaskManager registrations
Duplicate TaskManager registrations shouldn't be logged with Exceptions
in the ResourceManager. Duplicate registrations can happen if the
TaskManager sends out registration messages too fast when the actual
reply is not lost but still in transit.
The ResourceManager should simply acknowledge the duplicate
registrations, leaving it up to the JobManager to decide how to treat
the duplicate registrations (currently it will send an AlreadyRegistered
to the TaskManager).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mxm/flink FLINK-3981
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/2045.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 #2045
----
commit 8106ab460920ccea2a96e90c3a778d7acac0c672
Author: Maximilian Michels <m...@apache.org>
Date: 2016-05-27T19:02:38Z
[FLINK-3981] don't log duplicate TaskManager registrations
Duplicate TaskManager registrations shouldn't be logged with Exceptions
in the ResourceManager. Duplicate registrations can happen if the
TaskManager sends out registration messages too fast when the actual
reply is not lost but still in transit.
The ResourceManager should simply acknowledge the duplicate
registrations, leaving it up to the JobManager to decide how to treat
the duplicate registrations (currently it will send an AlreadyRegistered
to the TaskManager).
----
> Don't log duplicate TaskManager registrations as exceptions
> -----------------------------------------------------------
>
> Key: FLINK-3981
> URL: https://issues.apache.org/jira/browse/FLINK-3981
> Project: Flink
> Issue Type: Bug
> Components: ResourceManager
> Affects Versions: 1.1.0
> Reporter: Maximilian Michels
> Assignee: Maximilian Michels
> Priority: Minor
> Labels: test-stability
> Fix For: 1.1.0
>
>
> Duplicate TaskManager registrations shouldn't be logged with Exceptions in
> the ResourceManager. Duplicate registrations can happen if the TaskManager
> sends out registration messages too fast when the actual reply is not lost
> but still in transit.
> The ResourceManager should simply acknowledge the duplicate registrations,
> leaving it up to the JobManager to decide how to treat the duplicate
> registrations (currently it will send an AlreadyRegistered to the
> TaskManager).
> This change also affects our test stability because the Yarn tests check the
> logs for exceptions.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
