edoardocomar commented on PR #16329:
URL: https://github.com/apache/kafka/pull/16329#issuecomment-2173274444

   > @edoardocomar Thanks for the PR. The metric is currently consistent with 
`responseSendTimeHist`, which is updated regardless of whether a response is 
sent (e.g. including produce requests with acks=0 where there is no response). 
Similarly, the conversion metric represents conversion time across all 
responses at the moment. Should we keep it that way?
   
   We could keep that way and close the JIRA - I found it surprising to have a 
count of conversions which includes the no-conversions. Perhaps we need a small 
KIP to clarify and maybe change the semantics ?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to