[ 
https://issues.apache.org/jira/browse/FLINK-9187?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16532739#comment-16532739
 ] 

ASF GitHub Bot commented on FLINK-9187:
---------------------------------------

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

    https://github.com/apache/flink/pull/6184#discussion_r200122815
  
    --- Diff: 
flink-metrics/flink-metrics-prometheus/src/main/java/org/apache/flink/metrics/prometheus/PrometheusReporter.java
 ---
    @@ -20,76 +20,38 @@
     
     import org.apache.flink.annotation.PublicEvolving;
     import org.apache.flink.annotation.VisibleForTesting;
    -import org.apache.flink.metrics.CharacterFilter;
    -import org.apache.flink.metrics.Counter;
    -import org.apache.flink.metrics.Gauge;
    -import org.apache.flink.metrics.Histogram;
    -import org.apache.flink.metrics.Meter;
     import org.apache.flink.metrics.Metric;
     import org.apache.flink.metrics.MetricConfig;
    -import org.apache.flink.metrics.MetricGroup;
     import org.apache.flink.metrics.reporter.MetricReporter;
    -import org.apache.flink.runtime.metrics.groups.AbstractMetricGroup;
    -import org.apache.flink.runtime.metrics.groups.FrontMetricGroup;
     import org.apache.flink.util.NetUtils;
     import org.apache.flink.util.Preconditions;
     
    -import io.prometheus.client.Collector;
    -import io.prometheus.client.CollectorRegistry;
     import io.prometheus.client.exporter.HTTPServer;
     import org.slf4j.Logger;
     import org.slf4j.LoggerFactory;
     
     import java.io.IOException;
    -import java.util.AbstractMap;
    -import java.util.ArrayList;
    -import java.util.Arrays;
    -import java.util.Collections;
    -import java.util.HashMap;
     import java.util.Iterator;
    -import java.util.LinkedList;
    -import java.util.List;
    -import java.util.Map;
    -import java.util.regex.Pattern;
     
     /**
      * {@link MetricReporter} that exports {@link Metric Metrics} via 
Prometheus.
      */
     @PublicEvolving
    -public class PrometheusReporter implements MetricReporter {
    -   private static final Logger LOG = 
LoggerFactory.getLogger(PrometheusReporter.class);
    +public class PrometheusReporter extends AbstractPrometheusReporter {
    +   private final Logger log = 
LoggerFactory.getLogger(PrometheusReporter.class);
    --- End diff --
    
    remove this logger and use the logger defined in the parent class


> add prometheus pushgateway reporter
> -----------------------------------
>
>                 Key: FLINK-9187
>                 URL: https://issues.apache.org/jira/browse/FLINK-9187
>             Project: Flink
>          Issue Type: New Feature
>          Components: Metrics
>    Affects Versions: 1.4.2
>            Reporter: lamber-ken
>            Priority: Minor
>              Labels: features, pull-request-available
>             Fix For: 1.6.0
>
>
> make flink system can send metrics to prometheus via pushgateway.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to