[GitHub] flink pull request #4803: [FLINK-7813] [metrics] Replace MetricRegistryThrea...

2017-10-18 Thread zentol
Github user zentol closed the pull request at: https://github.com/apache/flink/pull/4803 ---

[GitHub] flink pull request #4803: [FLINK-7813] [metrics] Replace MetricRegistryThrea...

2017-10-18 Thread zentol
Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/4803#discussion_r145347968 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/metrics/MetricRegistry.java --- @@ -402,25 +401,4 @@ public void run() {

[GitHub] flink pull request #4803: [FLINK-7813] [metrics] Replace MetricRegistryThrea...

2017-10-17 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/4803#discussion_r145122089 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/metrics/MetricRegistry.java --- @@ -402,25 +401,4 @@ public void run() {

[GitHub] flink pull request #4803: [FLINK-7813] [metrics] Replace MetricRegistryThrea...

2017-10-11 Thread zentol
GitHub user zentol opened a pull request: https://github.com/apache/flink/pull/4803 [FLINK-7813] [metrics] Replace MetricRegistryThreadFactory ## What is the purpose of the change This PR modifies the MetricRegistry to use the ExecutorThreadFactory instead of its own