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

Ksolves commented on KAFKA-14348:
---------------------------------

[~cmccabe] It's an old ticket but would like to confirm if the change should be 
made in 
the`[BrokerServerMetrics.scala#L36-L39|https://github.com/apache/kafka/blob/3.7.0/core/src/main/scala/kafka/server/metadata/BrokerServerMetrics.scala#L36-L39]`?

```
private val batchProcessingTimeHistName = 
KafkaMetricsGroup.explicitMetricName("kafka.server",
    "BrokerMetadataListener",
      "{*}MetadataBatchProcessingTimeUs{*}",
    Collections.emptyMap())

```

> Consider renaming MetadataBatchProcessingTimeUs to 
> MetadataDeltaProcessingTimeUs
> --------------------------------------------------------------------------------
>
>                 Key: KAFKA-14348
>                 URL: https://issues.apache.org/jira/browse/KAFKA-14348
>             Project: Kafka
>          Issue Type: Improvement
>            Reporter: Colin McCabe
>            Assignee: Colin McCabe
>            Priority: Major
>              Labels: kip-500
>
> We should consider renaming kafka.server.MetadataBatchProcessingTimeUs to 
> kafka.server.MetadataDeltaProcessingTimeUs. The reason is because this metric 
> isn't the time to process a single batch, but the time to process a group of 
> batches given to us by the raft layer.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to