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

Jorge Esteban Quilcate Otoya commented on KAFKA-14953:
------------------------------------------------------

[~abhijeetkumar] thanks for working on this PR!

Looking into the metrics available, seems that we are missing 
`{color:#000000}RemoteWriteRequestsPerSec{color}` (there is 
`{color:#000000}RemoteWriteErrorPerSec{color}`and 
`{color:#000000}RemoteReadRequestsPerSec{color}`, I don't see why not adding 
it). I think we can make a small amend to the KIP, and ship ti on the same PR, 
wdyt? cc [~showuon] 

> Add metrics for tiered storage
> ------------------------------
>
>                 Key: KAFKA-14953
>                 URL: https://issues.apache.org/jira/browse/KAFKA-14953
>             Project: Kafka
>          Issue Type: Sub-task
>            Reporter: Divij Vaidya
>            Assignee: Abhijeet Kumar
>            Priority: Minor
>
> Not just for expired fetch. We also need to add all the metrics described in 
> KIP-405
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-405%3A+Kafka+Tiered+Storage#KIP405:KafkaTieredStorage-NewMetrics
>  
> ref: [https://github.com/apache/kafka/pull/13535#discussion_r1180286031] 



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

Reply via email to