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

    https://github.com/apache/flink/pull/5182#discussion_r157907331
  
    --- Diff: 
flink-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/internals/ShardConsumer.java
 ---
    @@ -96,6 +100,14 @@ protected ShardConsumer(KinesisDataFetcher<T> 
fetcherRef,
                                                        SequenceNumber 
lastSequenceNum,
                                                        KinesisProxyInterface 
kinesis) {
                this.fetcherRef = checkNotNull(fetcherRef);
    +           MetricGroup kinesisMetricGroup = fetcherRef.getRuntimeContext()
    +                   .getMetricGroup()
    +                   .addGroup("Kinesis");
    +           kinesisMetricGroup
    +                   .getAllVariables()
    +                   .put("<shard_id>", 
subscribedShard.getShard().getShardId());
    +
    +           kinesisMetricGroup.gauge("millisBehindLatest", (Gauge<Long>) () 
-> millisBehindLatest);
    --- End diff --
    
    the cast shouldn't be necessary


---

Reply via email to