GitHub user sihuazhou opened a pull request:
https://github.com/apache/flink/pull/5879
[FLINK-9215][resoucemanager] Only log the exception when log level is TRACE
to avoid confusing
## What is the purpose of the change
This PR fix the log in `SlotPool#releaseSlot()` to avoid confusing.
## Brief change log
- *Only log the exception when the log level is TRACE in
SlothPool#releaseSlot()*
- *dd double check in ZooKeeperLeaderRetrievalService#start() to avoid
race condition*
## Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
## Does this pull request potentially affect one of the following parts:
no
## Documentation
no
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sihuazhou/flink fixLogForTaskManagerReleasing
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/5879.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 #5879
----
commit 7afe861ce5161b1157e1889861aeaeeec2456181
Author: sihuazhou <summerleafs@...>
Date: 2018-04-19T08:13:58Z
Only log the exception when the log level is TRACE in
SlothPool#releaseSlot() to avoid confusing.
commit 06a6ece54b2720abd79d5c0673b12ff5e2e83054
Author: sihuazhou <summerleafs@...>
Date: 2018-04-19T08:16:33Z
[hotfix] Add double check in ZooKeeperLeaderRetrievalService#start() to
avoid race condition.
----
---
