Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6400#discussion_r204757730
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/legacy/metrics/JobVertexMetricsHandler.java
 ---
    @@ -57,4 +70,77 @@ public JobVertexMetricsHandler(Executor executor, 
MetricFetcher fetcher) {
                        ? task.metrics
                        : null;
        }
    +
    +   @Override
    +   protected String getRequestMetricsList(Map<String, String> queryParams) 
{
    +           String metricRequestsList = queryParams.get(PARAMETER_METRICS);
    +           String subtasksList = queryParams.get(SUB_TASKS);
    +           if (subtasksList == null || subtasksList.isEmpty()) {
    +                   return queryParams.get(PARAMETER_METRICS);
    --- End diff --
    
    return `super.getRequestMetricsList(queryParams)` instead


---

Reply via email to