[GitHub] flink pull request #2886: [FLINK-5179] Correct TM MetricRegistry and TmMG li...

2016-12-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/2886 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] flink pull request #2886: [FLINK-5179] Correct TM MetricRegistry and TmMG li...

2016-12-02 Thread zentol
Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/2886#discussion_r90653350 --- Diff: flink-runtime/src/main/scala/org/apache/flink/runtime/taskmanager/TaskManager.scala --- @@ -1046,11 +1046,10 @@ class TaskManager(

[GitHub] flink pull request #2886: [FLINK-5179] Correct TM MetricRegistry and TmMG li...

2016-12-02 Thread zentol
Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/2886#discussion_r90652945 --- Diff: flink-runtime/src/main/scala/org/apache/flink/runtime/taskmanager/TaskManager.scala --- @@ -1046,11 +1046,10 @@ class TaskManager(

[GitHub] flink pull request #2886: [FLINK-5179] Correct TM MetricRegistry and TmMG li...

2016-12-02 Thread uce
Github user uce commented on a diff in the pull request: https://github.com/apache/flink/pull/2886#discussion_r90651728 --- Diff: flink-runtime/src/main/scala/org/apache/flink/runtime/taskmanager/TaskManager.scala --- @@ -1046,11 +1046,10 @@ class TaskManager(

[GitHub] flink pull request #2886: [FLINK-5179] Correct TM MetricRegistry and TmMG li...

2016-11-28 Thread zentol
GitHub user zentol opened a pull request: https://github.com/apache/flink/pull/2886 [FLINK-5179] Correct TM MetricRegistry and TmMG lifecycle This PR corrects the life-cycle of the MetricRegistry and TaskManagerMetricGroup of the TaskManager. The MetricRegistry is no