Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/5055
---
Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5055#discussion_r152951899
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/job/metrics/AbstractMetricsHandler.java
---
@@ -0,0 +1,141 @@
+/*
+ *
Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/5055#discussion_r152917601
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/job/metrics/AbstractMetricsHandler.java
---
@@ -0,0 +1,141 @@
+/*
Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5055#discussion_r152837007
--- Diff:
flink-runtime/src/test/java/org/apache/flink/runtime/rest/messages/job/metrics/MetricsFilterParameterTest.java
---
@@ -0,0 +1,51 @@
+/*
+ *
Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5055#discussion_r152817079
--- Diff:
flink-runtime/src/test/java/org/apache/flink/runtime/rest/messages/job/metrics/MetricsFilterParameterTest.java
---
@@ -0,0 +1,51 @@
+/*
+ *
Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5055#discussion_r152804507
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/job/metrics/AbstractMetricsHandler.java
---
@@ -0,0 +1,141 @@
+/*
+ *
GitHub user GJL opened a pull request:
https://github.com/apache/flink/pull/5055
[FLINK-7718] [flip6] Add JobVertexMetricsHandler to DispatcherRestEndpoint
## Brief change log
- *Migrate logic in
`org.apache.flink.runtime.rest.handler.legacy.metrics.JobVertexMetricsHandler`